-
Notifications
You must be signed in to change notification settings - Fork 30
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
Demangle #123
Demangle #123
Conversation
I'm fine with demangling by default. |
Ok, looks like we have the same Nightly failures in https://github.com/tshort/StaticCompiler.jl/tree/rerun-ci (something to do with Nightly breaking LV / StrideArrays?) but I'll wait a day to see if it resolves itself If folks are good with it, I'll plan to release this as 0.4.12 and then switch to demangling by default as 0.5.0 (and switch over StaticTools and examples to match) |
For reference in case the logs disappear in the future, the failure on nightly here
vs on main
|
This attempts to re-implement name demangling in a consistent way across all methods, plus a bit of general cleanup
There are some failures on Nightly that I'm not sure the deal with yet, will have to check if they are from this PR or not..
We could also think about whether we want to turn on demangling by default -- would arguably more convenient and logical long-term, though possibly technically a breaking change?