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
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to