[PATCH] D71145: [InstCombine] Allow to limit the max number of iterations

[Jakub Kuderski via Phabricator via llvm-commits]

kuhar marked an inline comment as done and an inline comment as not done.
kuhar added inline comments.


================
Comment at: llvm/lib/Transforms/InstCombine/InstructionCombining.cpp:3685
+    : FunctionPass(ID), ExpensiveCombines(ExpensiveCombines),
+      MaxIterations(UINT_MAX) {
+  initializeInstructionCombiningPassPass(*PassRegistry::getPassRegistry());
----------------
spatel wrote:
> kuhar wrote:
> > Forgot to update the constant here. Will fix in the next revision.
> This is marked as 'Done' but UINT_MAX should be changed to InstCombineDefaultMaxIterations?
Yes. I have it fixed locally like you mentioned.


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D71145/new/

https://reviews.llvm.org/D71145