[PATCH] D71145: [InstCombine] Allow to limit the max number of iterations
Jakub Kuderski via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Tue Dec 17 10:35:57 PST 2019
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
More information about the llvm-commits
mailing list