[PATCH] D74476: [NFC][AggressiveInstCombine] Remove redundant std::max.

Ayman Musa via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Feb 12 03:46:18 PST 2020


aymanmus created this revision.
Herald added subscribers: llvm-commits, hiraditya.
Herald added a project: LLVM.

Remove unnecessary std::max call.
LGTM granted in D72837 <https://reviews.llvm.org/D72837>


https://reviews.llvm.org/D74476

Files:
  llvm/lib/Transforms/AggressiveInstCombine/TruncInstCombine.cpp


Index: llvm/lib/Transforms/AggressiveInstCombine/TruncInstCombine.cpp
===================================================================
--- llvm/lib/Transforms/AggressiveInstCombine/TruncInstCombine.cpp
+++ llvm/lib/Transforms/AggressiveInstCombine/TruncInstCombine.cpp
@@ -194,7 +194,7 @@
         unsigned IOpBitwidth = InstInfoMap.lookup(IOp).ValidBitWidth;
         if (IOpBitwidth >= ValidBitWidth)
           continue;
-        InstInfoMap[IOp].ValidBitWidth = std::max(ValidBitWidth, IOpBitwidth);
+        InstInfoMap[IOp].ValidBitWidth = ValidBitWidth;
         Worklist.push_back(IOp);
       }
   }


-------------- next part --------------
A non-text attachment was scrubbed...
Name: D74476.244120.patch
Type: text/x-patch
Size: 612 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20200212/56856c30/attachment.bin>


More information about the llvm-commits mailing list