[PATCH] D125264: [ConstraintElimination] Simplify ssub(a, b) if a s>=b && a s>=0.
Florian Hahn via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Mon May 9 13:17:47 PDT 2022
fhahn added a comment.
In D125264#3501773 <https://reviews.llvm.org/D125264#3501773>, @nikic wrote:
> Would it be sufficient to only replace the `m_ExtractValue<1>`? I believe that's what we usually do. (Would also work if only the extractvalue is dominated.)
And leave the removal of the actual intrinsic call to instcombine (or some other existing transform)? If that's what we do elsewhere I think it should work, the current placement in the regular optimization pipeline means we run instcombine shortly after ConstraintElimiantion.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D125264/new/
https://reviews.llvm.org/D125264
More information about the llvm-commits
mailing list