[llvm] [InstCombine] Allow freezing multiple operands (PR #154336)
Simon Pilgrim via llvm-commits
llvm-commits at lists.llvm.org
Tue Aug 19 07:15:29 PDT 2025
RKSimon wrote:
What caused so much trouble in #145939 is that SelectionDAG is terrible at topological combine order, which was resulting in the replaceUses calls causing infinite loops because the freeze was getting pushed through beyond the recursion depth that some other users (combined later) could see. Is that likely to happen in InstCombine?
I've started work on a much tamer version here #152107 - which relies on a separate fold of frozen/unfrozen versions of a node getting merged later on.
https://github.com/llvm/llvm-project/pull/154336
More information about the llvm-commits
mailing list