[PATCH] D43789: [ValueTracking] Teach cannotBeOrderedLessThanZeroImpl to look through ExtractElement.

Phabricator via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Feb 27 11:58:40 PST 2018


This revision was automatically updated to reflect the committed changes.
Closed by commit rL326237: [ValueTracking] Teach cannotBeOrderedLessThanZeroImpl to look through… (authored by ctopper, committed by ).

Repository:
  rL LLVM

https://reviews.llvm.org/D43789

Files:
  llvm/trunk/lib/Analysis/ValueTracking.cpp
  llvm/trunk/test/Transforms/InstSimplify/floating-point-arithmetic.ll


Index: llvm/trunk/lib/Analysis/ValueTracking.cpp
===================================================================
--- llvm/trunk/lib/Analysis/ValueTracking.cpp
+++ llvm/trunk/lib/Analysis/ValueTracking.cpp
@@ -2783,6 +2783,12 @@
     // Widening/narrowing never change sign.
     return cannotBeOrderedLessThanZeroImpl(I->getOperand(0), TLI, SignBitOnly,
                                            Depth + 1);
+  case Instruction::ExtractElement:
+    // Look through extract element. At the moment we keep this simple and skip
+    // tracking the specific element. But at least we might find information
+    // valid for all elements of the vector.
+    return cannotBeOrderedLessThanZeroImpl(I->getOperand(0), TLI, SignBitOnly,
+                                           Depth + 1);
   case Instruction::Call:
     const auto *CI = cast<CallInst>(I);
     Intrinsic::ID IID = getIntrinsicForCallSite(CI, TLI);
Index: llvm/trunk/test/Transforms/InstSimplify/floating-point-arithmetic.ll
===================================================================
--- llvm/trunk/test/Transforms/InstSimplify/floating-point-arithmetic.ll
+++ llvm/trunk/test/Transforms/InstSimplify/floating-point-arithmetic.ll
@@ -435,3 +435,17 @@
   %fabs = call float @llvm.fabs.f32(float %sqrt)
   ret float %fabs
 }
+
+define float @fabs_select_positive_constants_vector_extract(i32 %c) {
+; CHECK-LABEL: @fabs_select_positive_constants_vector_extract(
+; CHECK-NEXT:    [[CMP:%.*]] = icmp eq i32 [[C:%.*]], 0
+; CHECK-NEXT:    [[SELECT:%.*]] = select i1 [[CMP]], <2 x float> <float 1.000000e+00, float 1.000000e+00>, <2 x float> <float 2.000000e+00, float 2.000000e+00>
+; CHECK-NEXT:    [[EXTRACT:%.*]] = extractelement <2 x float> [[SELECT]], i32 0
+; CHECK-NEXT:    ret float [[EXTRACT]]
+;
+  %cmp = icmp eq i32 %c, 0
+  %select = select i1 %cmp, <2 x float> <float 1.0, float 1.0>, <2 x float> <float 2.0, float 2.0>
+  %extract = extractelement <2 x float> %select, i32 0
+  %fabs = call float @llvm.fabs.f32(float %extract)
+  ret float %fabs
+}


-------------- next part --------------
A non-text attachment was scrubbed...
Name: D43789.136126.patch
Type: text/x-patch
Size: 2040 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20180227/fc36e5ea/attachment.bin>


More information about the llvm-commits mailing list