[PATCH] D30350: [LSR] Add a cap for reassociation of AllFixupsOutsideLoop type LSRUse to protect compile time

[Davide Italiano via Phabricator via llvm-commits]

davide added inline comments.


================
Comment at: lib/Transforms/Scalar/LoopStrengthReduce.cpp:3429
+  // Arbitrarily choose a cap size of AddOps to protect compile time.
+  if (AddOps.size() >= 5 && LU.AllFixupsOutsideLoop)
+    return;
----------------
How did you chose this cap, BTW? 
Maybe we should do some measurements and pick a less arbitrary value?


Repository:
  rL LLVM

https://reviews.llvm.org/D30350