[llvm] [InstCombine] Preload DomConditionCache to reach fixed point when sinking (PR #170835)
Dominik Montada via llvm-commits
llvm-commits at lists.llvm.org
Fri Dec 5 05:46:29 PST 2025
gargaroff wrote:
> > We're hitting this bug in our downstream fork on real life content so our only option is to either increase the InstCombine iteration limit to 2 or to fix this bug.
>
> Then you are invoking InstCombine incorrectly. Production use must use `no-verify-fixpoint`, which is the default when creating an InstCombine instance via the C++ API. If you are creating it via a pass pipeline string, you need to specify this explicitly. `verify-fixpoint` is a testing facility and expected to fail.
Oh that's good to know, had no idea!
I'm gonna double check that in our fork, but would still be nice if we could fix this issue. I'm gonna check whether I can come up with a solution that is less impactful on compile time.
https://github.com/llvm/llvm-project/pull/170835
More information about the llvm-commits
mailing list