From: Matt Arsenault <Matthew.Arsenault@amd.com>
Date: Tue, 20 Jun 2023 00:47:29 +0000 (-0400)
Subject: llvm-reduce: Fix introducing invalid uses of intrinsics
X-Git-Tag: upstream/17.0.6~4456
X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=29ce3678c0188a4c9ee6576934c3e4abd72ff12d;p=platform%2Fupstream%2Fllvm.git

llvm-reduce: Fix introducing invalid uses of intrinsics
---

diff --git a/llvm/test/tools/llvm-reduce/operands-skip-intrinsics.ll b/llvm/test/tools/llvm-reduce/operands-skip-intrinsics.ll
new file mode 100644
index 0000000..c134c01
--- /dev/null
+++ b/llvm/test/tools/llvm-reduce/operands-skip-intrinsics.ll
@@ -0,0 +1,45 @@
+; RUN: llvm-reduce --abort-on-invalid-reduction --delta-passes=operands-skip --test FileCheck --test-arg --check-prefixes=INTERESTING --test-arg %s --test-arg --input-file %s -o %t
+; RUN: FileCheck --check-prefixes=CHECK %s < %t
+
+; Make sure address captures of intrinsics are not introduced.
+
+
+declare ptr @fptr()
+declare ptr @fptr.keep()
+declare ptr @llvm.intrin()
+declare ptr @llvm.intrin.keep()
+declare void @func(i32, ptr)
+
+declare ptr @llvm.intrin2()
+declare ptr @llvm.intrin.chain(ptr)
+
+; INTERESTING-LABEL: define void @caller(
+; INTERESTING: call void @func(i32 1
+; INTERESTING: call void @func(i32 3
+; INTERESTING: call void @func(i32 4
+; INTERESTING: call void @func(i32 5
+
+; CHECK: %intrin.ptr.keep = call ptr @llvm.intrin.keep()
+
+; CHECK: call void @func(i32 0, ptr @fptr)
+; CHECK: call void @func(i32 1, ptr @fptr.keep)
+; CHECK: call void @func(i32 2, ptr %intrin.ptr)
+; CHECK: call void @func(i32 3, ptr %intrin.ptr.keep)
+; CHECK: call void @func(i32 4, ptr %intrin.ptr.keep)
+; CHECK: call void @func(i32 5, ptr %chained.ptr)
+define void @caller() {
+  %func.ptr = call ptr @fptr()
+  %func.ptr.keep = call ptr @fptr.keep()
+  %intrin.ptr = call ptr @llvm.intrin()
+  %intrin.ptr.keep = call ptr @llvm.intrin.keep()
+  %gep = getelementptr i8, ptr %intrin.ptr.keep, i64 128
+  %chained.ptr = call ptr @llvm.intrin2()
+  %chain.ptr = call ptr @llvm.intrin.chain(ptr %chained.ptr)
+  call void @func(i32 0, ptr %func.ptr)
+  call void @func(i32 1, ptr %func.ptr.keep)
+  call void @func(i32 2, ptr %intrin.ptr)
+  call void @func(i32 3, ptr %intrin.ptr.keep)
+  call void @func(i32 4, ptr %gep)
+  call void @func(i32 5, ptr %chain.ptr)
+  ret void
+}
diff --git a/llvm/tools/llvm-reduce/deltas/ReduceOperandsSkip.cpp b/llvm/tools/llvm-reduce/deltas/ReduceOperandsSkip.cpp
index a634c3a..52679f8 100644
--- a/llvm/tools/llvm-reduce/deltas/ReduceOperandsSkip.cpp
+++ b/llvm/tools/llvm-reduce/deltas/ReduceOperandsSkip.cpp
@@ -163,6 +163,12 @@ opportunities(Function &F,
         if (OpVal->getType() != V->getType())
           return true;
 
+        // Do not introduce address captures of intrinsics.
+        if (Function *F = dyn_cast<Function>(V)) {
+          if (F->isIntrinsic())
+            return true;
+        }
+
         // Only consider candidates that are "more reduced" than the original
         // value. This explicitly also rules out candidates with the same
         // reduction power. This is to ensure that repeated invocations of this