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

Long build times #512

Open
orion160 opened this issue Aug 11, 2024 · 4 comments
Open

Long build times #512

orion160 opened this issue Aug 11, 2024 · 4 comments
Labels
component: sparta Issue is related to sparta framework enhancement Enhancement or request

Comments

@orion160
Copy link

No description provided.

@orion160
Copy link
Author

@klingaard Can I address this and reformat cmake build system to be more target centric. And to provide better library handling?

@klingaard
Copy link
Member

Sure. However, I'd like to understand what you'd like to change. Specifically,

  1. Long build times of what? Sparta or your target simulator?
  2. What do you mean by "more target specific"?
  3. What do you mean by "better library handling"?

@klingaard klingaard added enhancement Enhancement or request component: sparta Issue is related to sparta framework labels Aug 14, 2024
@orion160
Copy link
Author

It would be to do a minor refactor for structuring the build system on cmake targets. And to provide a SpartaConfig.cmake, which I see that the one used in the repo serves other purpose..

So to add sparta would be in the more standarized way with SPARTA_ROOT as a cache variable

Mmmmm and on the buld times i have to check out and see if there can be some improvements. Although great part of it is by having LTO enabled

@klingaard
Copy link
Member

I've made some advancements in "compiler time" with Sparta by moving code from header files to source files based on using clang's time reports: https://aras-p.info/blog/2019/01/12/Investigating-compile-times-and-Clang-ftime-report/

You can use chrome's profiler view on the report: https://aras-p.info/blog/2017/01/23/Chrome-Tracing-as-Profiler-Frontend/

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
component: sparta Issue is related to sparta framework enhancement Enhancement or request
Projects
None yet
Development

No branches or pull requests

2 participants