diff options
| author | Yingwei Zheng <dtcxzyw2333@gmail.com> | 2024-03-26 20:56:16 +0800 |
|---|---|---|
| committer | GitHub <noreply@github.com> | 2024-03-26 20:56:16 +0800 |
| commit | caa225825075237cd6b15757476dd1999bec290b (patch) | |
| tree | 3b257ca94019f1cefd050c6ce55c9e5b847d0a49 /llvm/lib/IR/Constants.cpp | |
| parent | feebcd65fb7e0534f5219e05432a05e45aa8cd2a (diff) | |
| download | llvm-caa225825075237cd6b15757476dd1999bec290b.zip llvm-caa225825075237cd6b15757476dd1999bec290b.tar.gz llvm-caa225825075237cd6b15757476dd1999bec290b.tar.bz2 | |
[LLVM] Remove nuw neg (#86295)
This patch removes APIs that creating NUW neg. It is a trivial case
because `sub nuw 0, X` always gets simplified into zero.
I believe there is no optimization opportunities in the real-world
applications that we can take advantage of the nuw flag.
Motivated by
https://github.com/llvm/llvm-project/pull/84792#discussion_r1524891134.
Compile-time improvement:
https://llvm-compile-time-tracker.com/compare.php?from=d1f182c895728d89c5c3d198b133e212a5d9d4a3&to=da7b7478b7cbb32c09d760f6b8d0e67901e0d533&stat=instructions:u
Diffstat (limited to 'llvm/lib/IR/Constants.cpp')
| -rw-r--r-- | llvm/lib/IR/Constants.cpp | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/llvm/lib/IR/Constants.cpp b/llvm/lib/IR/Constants.cpp index c17419b..a5fb497 100644 --- a/llvm/lib/IR/Constants.cpp +++ b/llvm/lib/IR/Constants.cpp @@ -2520,10 +2520,10 @@ Constant *ConstantExpr::getShuffleVector(Constant *V1, Constant *V2, return pImpl->ExprConstants.getOrCreate(ShufTy, Key); } -Constant *ConstantExpr::getNeg(Constant *C, bool HasNUW, bool HasNSW) { +Constant *ConstantExpr::getNeg(Constant *C, bool HasNSW) { assert(C->getType()->isIntOrIntVectorTy() && "Cannot NEG a nonintegral value!"); - return getSub(ConstantInt::get(C->getType(), 0), C, HasNUW, HasNSW); + return getSub(ConstantInt::get(C->getType(), 0), C, /*HasNUW=*/false, HasNSW); } Constant *ConstantExpr::getNot(Constant *C) { |