[PATCH] D106850: [InstCombine] Simplify llvm.vscale when vscale_range attribute exists

JunMa via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Jul 28 04:24:11 PDT 2021


junparser updated this revision to Diff 362330.
junparser added a comment.

Address comment. Move change to InstSimplify Pass


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D106850/new/

https://reviews.llvm.org/D106850

Files:
  llvm/lib/Analysis/InstructionSimplify.cpp
  llvm/test/Transforms/InstSimplify/fold-vscale.ll


Index: llvm/test/Transforms/InstSimplify/fold-vscale.ll
===================================================================
--- /dev/null
+++ llvm/test/Transforms/InstSimplify/fold-vscale.ll
@@ -0,0 +1,44 @@
+; NOTE: Assertions have been autogenerated by utils/update_test_checks.py
+; RUN: opt < %s -instsimplify -S | FileCheck %s
+
+define i64 @vscale_i64() #1 {
+; CHECK-LABEL: @vscale_i64(
+; CHECK-NEXT:    ret i64 16
+;
+  %1 = call i64 @llvm.vscale.i64()
+  %out = shl i64 %1, 4
+  ret i64 %out
+}
+
+define i32 @vscale_i32() #2 {
+; CHECK-LABEL: @vscale_i32(
+; CHECK-NEXT:    ret i32 8
+;
+  %1 = call i32 @llvm.vscale.i32()
+  %out = shl i32 %1, 2
+  ret i32 %out
+}
+
+define i64 @vscale_i64_diff() #3 {
+; CHECK-LABEL: @vscale_i64_diff(
+; CHECK-NEXT:    [[TMP1:%.*]] = call i64 @llvm.vscale.i64()
+; CHECK-NEXT:    [[OUT:%.*]] = shl i64 [[TMP1]], 4
+; CHECK-NEXT:    ret i64 [[OUT]]
+;
+  %1 = call i64 @llvm.vscale.i64()
+  %out = shl i64 %1, 4
+  ret i64 %out
+}
+
+; Function Attrs: nofree nosync nounwind readnone willreturn
+declare i64 @llvm.vscale.i64() #0
+
+; Function Attrs: nofree nosync nounwind readnone willreturn
+declare i32 @llvm.vscale.i32() #0
+
+attributes #0 = { nofree nosync nounwind readnone willreturn }
+attributes #1 = { mustprogress nofree nosync nounwind uwtable vscale_range(1,1) }
+attributes #2 = { mustprogress nofree nosync nounwind uwtable vscale_range(2,2) }
+attributes #3 = { mustprogress nofree nosync nounwind uwtable vscale_range(2,4) }
+
+
Index: llvm/lib/Analysis/InstructionSimplify.cpp
===================================================================
--- llvm/lib/Analysis/InstructionSimplify.cpp
+++ llvm/lib/Analysis/InstructionSimplify.cpp
@@ -5772,13 +5772,29 @@
 
 static Value *simplifyIntrinsic(CallBase *Call, const SimplifyQuery &Q) {
 
-  // Intrinsics with no operands have some kind of side effect. Don't simplify.
   unsigned NumOperands = Call->getNumArgOperands();
-  if (!NumOperands)
-    return nullptr;
-
   Function *F = cast<Function>(Call->getCalledFunction());
   Intrinsic::ID IID = F->getIntrinsicID();
+
+  // Most of the intrinsics with no operands have some kind of side effect.
+  // Don't simplify.
+  if (!NumOperands) {
+    switch (IID) {
+    case Intrinsic::vscale: {
+      auto Attr = Call->getFunction()->getFnAttribute(Attribute::VScaleRange);
+      if (!Attr.isValid())
+        return nullptr;
+      unsigned MinSVEVectorSize, MaxSVEVectorSize;
+      std::tie(MinSVEVectorSize, MaxSVEVectorSize) = Attr.getVScaleRangeArgs();
+      if (MinSVEVectorSize == MaxSVEVectorSize && MaxSVEVectorSize != 0)
+        return ConstantInt::get(F->getReturnType(), MinSVEVectorSize);
+      return nullptr;
+    }
+    default:
+      return nullptr;
+    }
+  }
+
   if (NumOperands == 1)
     return simplifyUnaryIntrinsic(F, Call->getArgOperand(0), Q);
 


-------------- next part --------------
A non-text attachment was scrubbed...
Name: D106850.362330.patch
Type: text/x-patch
Size: 2850 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20210728/74664e4a/attachment.bin>


More information about the llvm-commits mailing list