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

[Sanjoy Das via Phabricator via llvm-commits]

sanjoy added inline comments.


================
Comment at: lib/Transforms/Scalar/LoopStrengthReduce.cpp:3430
+  if (AddOps.size() >= 5 && LU.AllFixupsOutsideLoop)
+    return;
+
----------------
I'm not too familiar with LSR, but this looks pretty ad-hoc -- why can't the same compile-time blowup happen for an `LSRUse` with `AllFixupsOutsideLoop` = `false`?


Repository:
  rL LLVM

https://reviews.llvm.org/D30350