[clang] [time-trace] Add a new time trace scope variable named "ParseDeclarationOrFunctionDefinition". (PR #65268)

Nikita Popov via cfe-commits cfe-commits at lists.llvm.org
Tue Oct 31 02:50:40 PDT 2023


nikic wrote:

I've reverted this change because it causes a large compile-time regression: http://llvm-compile-time-tracker.com/compare.php?from=61b9176cf70444c54f3ac6eebd82fc9ffd69944d&to=33b85867e30e1adc2ff2173039c199b81c10f52b&stat=instructions:u

>From a quick glance at your implementation, I suspect the issues is that the used strings may be expensive to compute. TimeTraceScope has an overload accepting a closure that calculates the detail string for such cases.

https://github.com/llvm/llvm-project/pull/65268


More information about the cfe-commits mailing list