[PATCH] D128097: [Clang] Fix compile time regression caused by D126061.

Nikita Popov via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Thu Jun 23 08:17:21 PDT 2022

nikic added a comment.

@mboehme Cycle counts are very noisy, and it's pretty much impossible to determine whether they changed by looking at a single commit, unless the differences are huge (like 10%). In this case, the commit got "lucky" and the next commit goes back to the previous level. This is why I only look at instruction counts.

And yes, there may be environment differences, either due to system headers, or due to the used host compiler (which is GCC 9.3 for llvm-compile-time-tracker).

As this patch did mitigate half the regression, and the regression wasn't particularly large in the first place, I personally wouldn't bother spending too much time to investigate this further.

  rG LLVM Github Monorepo



More information about the cfe-commits mailing list