aras-p / ClangBuildAnalyzer

Clang build analysis tool using -ftime-trace
The Unlicense
995 stars 64 forks source link

[#21] Fix crash at the cost of some correctness #24

Closed ben-craig closed 4 years ago

ben-craig commented 4 years ago

Band-aid for issue #21. Will no longer crash, but will not collapse some names for template analysis, and mis-collapse others.

At present, this only seems to affect "Function sets that took longest to compile / optimize". So it is possible that users will still get oddly collapsed names in those reports. The more likely outcome is that the mis-collapsed names won't get an opportunity to merge with other similar templates, so their score won't even show up in the list, even if it was part of a very expensive set of templates to optimize.