[PATCH] D3127: [ScalarEvolution]Fix PR18607 resulting in long compilation time and memory usage

David Majnemer via llvm-commits llvm-commits at lists.llvm.org
Wed Jul 27 20:29:29 PDT 2016


majnemer added a subscriber: majnemer.

================
Comment at: lib/Analysis/ScalarEvolution.cpp:1967
@@ +1966,3 @@
+      // Add a threshold to check the same.
+      if((Ops.size() + Mul->getNumOperands() -1) > 512)
+        break;
----------------
mehdi_amini wrote:
> 512 is you threshold right? Usually I believe we use a cl::opt with a default value for such things.
Formatting.


https://reviews.llvm.org/D3127





More information about the llvm-commits mailing list