-
-
Notifications
You must be signed in to change notification settings - Fork 76
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
Parallel simulations #507
Comments
The problem above where the object file names collide is easy enough to fix by adding e.g. the PID to the file names, but then there's another problem where elaboration generates a shared library Would it work for you if there was a
Which would produce separate shared libs for each elaboration. Then you could go back later and execute, say |
Another alternative is to add an elaboration option |
In my case I always re-elaborates each time and do elab+run as this will be used in a CI setup, so the name option is not as useful for me. An option for the separate elab and run crowd could be to somehow hash the generics provided and add them to the name, and then require the generics to also be present for the run command (unless you do elab+run). Run would then check if the unit with the specific generics has already been elaborated before running. |
I've added the
As a bonus elaboration will be slightly quicker as it's not saving the design hierarchy to disk any more. |
That was in commit c684f67, I didn't tag it properly. |
Possibly related, but when I kill NVC now with CTRL-C in windows I get
Except for that the parallel simulation works fine now. |
The regression with ctrl-c should be fixed now, but I'm not able to test on Windows at the moment. The "cannot remove" warning is unfortunate but I'm not sure what to do about that: Windows cannot remove files that are opened by any process, and it's a bit tricky to unload the DLL from the ctrl-c handler. |
Ctrl-C is fixed now. I do not get a "cannot remove" warning when using Ctrl-C while running without the |
I run tests of the same testcase with different generics in parallel from the same library. This seems to create some file and linking issues, but maybe this is not supposed to be possible.
The text was updated successfully, but these errors were encountered: