Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jeffnye gh/fsl submodule #175

Conversation

jeffnye-gh
Copy link
Collaborator

convert fusion support to sub-module fsl.

remove the old fusion dir. Add a cmake var so fsl uses olympia/mavis when it is instantiated as a submodule.

@@ -1,7 +1,7 @@
// <Decode.cpp> -*- C++ -*-

#include "Decode.hpp"
#include "fusion/FusionTypes.hpp"
#include "fsl_api/FusionTypes.h"
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Curious, why the change in name?

Copy link
Collaborator Author

@jeffnye-gh jeffnye-gh Jun 17, 2024

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

In the early days fusion was accurate, I have cases where I want to apply the language to fracture, some RV extensions and for vector.

I chose fsl_api and fsl_interp to separate the namespaces.

It could have been fsl and fsl_interp for the symmetry and to quiet my OCD.

Olympia is the lead use case, so if you prefer I can change fsl_api to fsl.

Jeff Nye added 2 commits July 25, 2024 11:33
@jeffnye-gh
Copy link
Collaborator Author

I pushed an update to correct the conflicts. re-requesting review

The push did not trigger the git runner, i do not know if it is supposed to for this change.

@jeffnye-gh jeffnye-gh merged commit 3313738 into riscv-software-src:master Jul 30, 2024
5 checks passed
@jeffnye-gh jeffnye-gh deleted the jeffnye-gh/fsl_submodule branch September 6, 2024 21:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants