[PATCH] D30280: [InstCombine] don't try SimplifyDemandedInstructionBits from zext/sext because it's slow and unnecessary

Sanjay Patel via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Feb 24 07:30:29 PST 2017


This revision was automatically updated to reflect the committed changes.
Closed by commit rL296129: [InstCombine] don't try SimplifyDemandedInstructionBits from zext/sext becauseā€¦ (authored by spatel).

Changed prior to commit:
  https://reviews.llvm.org/D30280?vs=89437&id=89670#toc

Repository:
  rL LLVM

https://reviews.llvm.org/D30280

Files:
  llvm/trunk/lib/Transforms/InstCombine/InstCombineCasts.cpp


Index: llvm/trunk/lib/Transforms/InstCombine/InstCombineCasts.cpp
===================================================================
--- llvm/trunk/lib/Transforms/InstCombine/InstCombineCasts.cpp
+++ llvm/trunk/lib/Transforms/InstCombine/InstCombineCasts.cpp
@@ -838,11 +838,6 @@
   if (Instruction *Result = commonCastTransforms(CI))
     return Result;
 
-  // See if we can simplify any instructions used by the input whose sole
-  // purpose is to compute bits we don't care about.
-  if (SimplifyDemandedInstructionBits(CI))
-    return &CI;
-
   Value *Src = CI.getOperand(0);
   Type *SrcTy = Src->getType(), *DestTy = CI.getType();
 
@@ -1124,11 +1119,6 @@
   if (Instruction *I = commonCastTransforms(CI))
     return I;
 
-  // See if we can simplify any instructions used by the input whose sole
-  // purpose is to compute bits we don't care about.
-  if (SimplifyDemandedInstructionBits(CI))
-    return &CI;
-
   Value *Src = CI.getOperand(0);
   Type *SrcTy = Src->getType(), *DestTy = CI.getType();
 


-------------- next part --------------
A non-text attachment was scrubbed...
Name: D30280.89670.patch
Type: text/x-patch
Size: 1018 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20170224/f2d9a83a/attachment.bin>


More information about the llvm-commits mailing list