[PATCH] D114095: [clang][lex] Include tracking: simplify and move to preprocessor

Volodymyr Sapsai via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Fri Dec 10 17:27:20 PST 2021


vsapsai added a comment.

Thanks for measuring the .pcm impact! And I appreciate including the trunk baseline.

The results aren't exactly the same I've seen before, so please forgive me my suspiciousness. When you mention

> current trunk with this patch

do you mean "just this D114095 <https://reviews.llvm.org/D114095> patch"? Or were you testing with D112915 <https://reviews.llvm.org/D112915> applied? Because I was testing with per-submodule tracking applied. I am going to check again but it would be useful to know if you were comparing with per-submodule tracking (D112915 <https://reviews.llvm.org/D112915>) or not.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D114095/new/

https://reviews.llvm.org/D114095



More information about the cfe-commits mailing list