[PATCH] D34056: Tail merge size

Jessica Paquette via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Jun 19 09:04:21 PDT 2017


paquette requested changes to this revision.
paquette added a comment.

I agree with Davide on this. I think we really need to have a good reason to change thresholds like these; the impact of changing thresholds can be rather unpredictable. I'd like to understand why changing this threshold

(1) Fixes the problem
(2) Won't (likely) have a negative impact on other tests

I think per-test measurements + flags used etc. would help here.


https://reviews.llvm.org/D34056





More information about the llvm-commits mailing list