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

Sanjoy Das via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Jun 20 22:28:42 PDT 2017


sanjoy added a comment.

32 seems like a sane default value, but where exactly do we get stuck here?  If there are O(N^2) or O(N^3) algorithms that are not super difficult to remove, we should just remove them instead of working around them like this.


https://reviews.llvm.org/D34397





More information about the llvm-commits mailing list