We already support SGE, so the same logic should hold for SLE with
the LHS and RHS swapped.
I didn't see this in the wild. Just happened to walk past this code
and thought it was odd that it was asymmetric in what condition
codes it handled.
Paths
| Differential D131805
[InstSimplify] sle on i1 also encodes implication ClosedPublic Authored by craig.topper on Aug 12 2022, 1:35 PM.
Details Summary We already support SGE, so the same logic should hold for SLE with I didn't see this in the wild. Just happened to walk past this code
Diff Detail
Event TimelineThis revision is now accepted and ready to land.Aug 15 2022, 8:00 AM Closed by commit rGef8c34e9543b: [InstSimplify] sle on i1 also encodes implication (authored by craig.topper). · Explain WhyAug 15 2022, 8:27 AM This revision was automatically updated to reflect the committed changes.
Revision Contents
Diff 452293 llvm/lib/Analysis/InstructionSimplify.cpp
llvm/test/Transforms/InstSimplify/implies.ll
|
clang-format not found in user’s local PATH; not linting file.