Demangling compiled C++ function names so that it makes the data more readable in analyzing platform.
Details
Diff Detail
Unit Tests
Event Timeline
Thanks, looks good (after reformatting suggested by lint).
Add @aras-p: hi, Aras, just to warn you about output format changing, which may be sensitive for tools like ClangBuildAnalyzer.
- Legacy PM is dead. Why change it? What about NewPM?
- Is this really an UX improvement? Now instead you can't easily find the LLVM IR function. There is a demangler command, but is there a mangler?
I agree with Roman. Those that want demangled names can just pipe through c++filt, but it's hard to go in the other direction, and the mangled names are easier to copy/paste/integrate with other tooling.
Thanks, sorry for the delayed reply.
Yes, fair point. How about adding an option to determine whether to demangle the function name or not?
Currently, I just want to make the output data generated by ftime-trace more readable in Chrome.
Sorry for delayed response. Thanks for your review, what is the follow-up of this patch? Any other thoughts?
clang-format: please reformat the code