[llvm] [NFC] Cleanup in ADT and Analysis headers. (PR #104484)

Jay Foad via llvm-commits llvm-commits at lists.llvm.org
Fri Aug 16 02:15:42 PDT 2024


jayfoad wrote:

> For reference: [compile-time-tracker run](https://llvm-compile-time-tracker.com/compare.php?from=141536544f4ec1d1bf24256157f4ff1a3bc07dae&to=9f78f2041a62ec3e5c2614ab5df66e8eb83ac9a2&stat=instructions:u) with the change.

Thanks, but that is measuring the speed of the generated LLVM binary. For a change like this, it would be more interesting to see whether the time taken to compile LLVM gets slower or (hopefully) faster. Or like I said before, you could use some other metric like the total number of preprocessed source lines.

The patch itself looks good to me but I only skimmed it.

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


More information about the llvm-commits mailing list