[llvm] [InstSimplify] Fix simplifyAndOrWithICmpEq with undef refinement (PR #98898)
Nikita Popov via llvm-commits
llvm-commits at lists.llvm.org
Mon Jul 15 07:37:43 PDT 2024
nikic wrote:
> re:
>
> > Finally, make sure that places where AllowRefinement=false also use Q.getWithoutUndef(). I don't have a specific test case for this (the original one does not work because we don't simplify selects with constant condition in this mode in the first place) but this seems like the correct thing to do to be conservative.
>
> What do you mean you don't have a test for this? Its a literal assert in the code? Do you mean you're not sure if this might trip?
>
> Otherwise this seems sensible.
What I meant is that I don't have a test case what shows an undef miscompile for users of simplifyWithOpReplaced that use AllowRefinement=false.
https://github.com/llvm/llvm-project/pull/98898
More information about the llvm-commits
mailing list