[PATCH] D39485: Extend SpecialCaseList to allow users to blame matches on entries in the file.

Peter Collingbourne via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Oct 31 18:14:01 PDT 2017


pcc added a comment.

I'd be curious about what sort of perf impact (positive or negative) comes from splitting the special case list into multiple regexes, as this part of the compiler has historically been perf sensitive. It may be that the trigram filter (which is relatively new) makes the impact relatively insignificant.

Perhaps a good way to measure would be to compile a relatively large Chromium TU with/without this patch.


https://reviews.llvm.org/D39485





More information about the llvm-commits mailing list