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

handcar_cosim.so is too big #72

Open
hanfeng0114 opened this issue May 30, 2023 · 3 comments
Open

handcar_cosim.so is too big #72

hanfeng0114 opened this issue May 30, 2023 · 3 comments

Comments

@hanfeng0114
Copy link
Contributor

Currently handcar_cosim.so is pushed into the force-riscv repo directly. The binary file is too big.
Start this issue to discuss possible maintenance policy like submodule?
Call for more discussions.

@MikeOpenHWGroup
Copy link
Member

Yes, 97M is a little big. 😄

Perhaps we can create a Makefile rule to allow users to compile it from source? We do that for the dpi_dasm shared object in CORE-V-VERIF. (Although because that so is only 9.7M we also include it in the repo.)

@hanfeng0114
Copy link
Contributor Author

Start with some possible solutions:
1.google-repo (one multi-repo management solution which is used for android project maintenance), which is T-HEAD XUANTIE's inhouse solution
2.git submodule: an easy solution to attach one repo in another repo
3.mix with git submodule and google-repo: we may include other repo like riscv-opcode which helps spike development

@hanfeng0114
Copy link
Contributor Author

Will start a meeting recently about this issue. @MikeOpenHWGroup @DBees @rchu-futurewei

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

No branches or pull requests

2 participants