[PATCH] D154579: [InstCombine] Only perform one iteration

Mon Jul 31 12:51:00 PDT 2023

nikic added inline comments.

================
Comment at: llvm/lib/Passes/PassRegistry.def:511
                           "no-use-loop-info;use-loop-info;"
                           "max-iterations=N"
                           )
----------------
bjope wrote:
> bjope wrote:
> > bjope wrote:
> > > We should add "no-verify-fixpoint;verify-fixpoint;" here, right?
> > Also noticed that `instcombine<verify-fixpoint>` will be tricky to use in fuzzy testing with random pipelines. So I think we will avoid that.
> I solved this in https://reviews.llvm.org/rG5fbee1c6e300eee9ce9d18275bf8a6de0a22ba59
Thank you! And yes, for fuzzing purposes, `instcombine<no-verify-fixpoint>` should be used.

Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D154579