[llvm] [ConstraintElimination] Extend unsigned-to-signed fact transfer (PR #66173)
Florian Hahn via llvm-commits
llvm-commits at lists.llvm.org
Tue Sep 19 04:04:41 PDT 2023
================
@@ -748,11 +749,22 @@ void ConstraintInfo::transferToOtherSystem(
default:
break;
case CmpInst::ICMP_ULT:
- // If B is a signed positive constant, A >=s 0 and A <s B.
+ case CmpInst::ICMP_ULE:
+ // If B is a signed positive constant, A >=s 0 and A <s (or <=s) B.
----------------
fhahn wrote:
I think adding a then would make the sentence easier to parse.
```suggestion
// If B is a signed positive constant, then A >=s 0 and A <s (or <=s) B.
```
https://github.com/llvm/llvm-project/pull/66173
More information about the llvm-commits
mailing list