-
-
Notifications
You must be signed in to change notification settings - Fork 5.5k
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
Unable to find compatible target in system image.
when using valgrind
#27131
Comments
This is most likely do to valgrind intercepting [1] https://stackoverflow.com/questions/48697579/valgrind-changes-cpuid-value |
Both valgrind and rr mask out cpuid values we don't really care about. We should adjust the processor detection code to ignore those. In the meantime @JeffBezanson you can comment out that error message and set |
Yes, that's what I do now. We should implement and/or document a real solution. |
These are probably not used by LLVM so ignoreing these now is OK but I don't think that's a good idea in the long term since user code might want to use them after the CPU detection API is exposed to julia code. The current way I deal with this is to make sure I have a target compiled that doesn't have all those features enabled. Other "more out of the box" solutions that I've considered (but decided to not do any until someone actual complain since none of them are perfect) includes.
|
At least for the purposes of loading the sysimg we should ignore all the instructions llvm doesn't use. If an when user code starts using the cpuid code, we can come up with a better solution. |
I'd consider this developer-only, so we don't need to make any drastic changes like changing the default target to support it. Even rebuilding the system image would be fine. A hidden command line option would be fine too. Mostly just needs to be documented in devdocs/valgrind.md. |
In case somebody is having a hard time running Julia with Valgrind, rebuilding the sysimage for the |
If there's a trick to making this work, it should be added to the valgrind devdocs.
The text was updated successfully, but these errors were encountered: