[llvm] [ConstraintElim] Fix poison check before adding intrinsic facts (PR #136291)
Nikita Popov via llvm-commits
llvm-commits at lists.llvm.org
Wed Apr 30 06:08:08 PDT 2025
nikic wrote:
> > Extending this check to abs is problematic due to the interaction with the int_min_is_poison flag.
>
> Then there's no functional change in this patch.
Right. I think it's okay to add the poison check workaround for usub_sat to unblock the patch stack, but I don't think we should touch abs until we can handle this properly.
https://github.com/llvm/llvm-project/pull/136291
More information about the llvm-commits
mailing list