[PATCH] D19459: Cleanup redundant expression in InstCombineAndOrXor.

Etienne Bergeron via cfe-commits cfe-commits at lists.llvm.org
Sat Apr 23 12:10:20 PDT 2016


etienneb created this revision.
etienneb added a reviewer: rnk.
etienneb added a subscriber: cfe-commits.

The expression is redundant on both side of operator |.

detected by : http://reviews.llvm.org/D19451

http://reviews.llvm.org/D19459

Files:
  lib/Transforms/InstCombine/InstCombineAndOrXor.cpp

Index: lib/Transforms/InstCombine/InstCombineAndOrXor.cpp
===================================================================
--- lib/Transforms/InstCombine/InstCombineAndOrXor.cpp
+++ lib/Transforms/InstCombine/InstCombineAndOrXor.cpp
@@ -465,11 +465,9 @@
   if (CCst && CCst->isZero()) {
     // if C is zero, then both A and B qualify as mask
     result |= (icmp_eq ? (FoldMskICmp_Mask_AllZeroes |
-                          FoldMskICmp_Mask_AllZeroes |
                           FoldMskICmp_AMask_Mixed |
                           FoldMskICmp_BMask_Mixed)
                        : (FoldMskICmp_Mask_NotAllZeroes |
-                          FoldMskICmp_Mask_NotAllZeroes |
                           FoldMskICmp_AMask_NotMixed |
                           FoldMskICmp_BMask_NotMixed));
     if (icmp_abit)


-------------- next part --------------
A non-text attachment was scrubbed...
Name: D19459.54786.patch
Type: text/x-patch
Size: 814 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20160423/825f6487/attachment.bin>


More information about the cfe-commits mailing list