[PATCH] D53245: Teach the DominatorTree fallback to recalculation when applying updates to speedup JT (PR37929)
Jakub Kuderski via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Mon Oct 15 07:58:11 PDT 2018
kuhar added a comment.
In https://reviews.llvm.org/D53245#1265405, @NutshellySima wrote:
> For `k>n` you mentioned, I think you mean `k>n/α` (α is a constant) because for the reproducer in 37929, the minimum value of α to make the time consumed by clang acceptable is about 3 (α should definitely be larger than 3). But selecting a higher value like "75" can also boost other inputs more (about 2x~4.5x on `clang*.bc`s).
OK, makes sense if that the case in the bug report.
Would you be able to produce some numbers for clang for a heuristic like this:
`if (n > 100) k * a >n; else if (k > n)`
with different `a`s?
https://reviews.llvm.org/D53245
More information about the llvm-commits
mailing list