[PATCH] D34397: [SCEV] Make MulOpsInlineThreshold lower to avoid excessive compilation time

Max Kazantsev via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Jun 20 05:29:45 PDT 2017


mkazantsev added a comment.

It is not clear whether we should also decrease AddOpsInlineThreshold or not. On one hand, in theory the same situation is possible there. On another hand, AddExprs with repeating operands are likely to merge them with Mul, so it may be a challenging task to construct a test where it plays a role. I will do some more investigation about that.


https://reviews.llvm.org/D34397





More information about the llvm-commits mailing list