[PATCH] D153313: llvm-reduce: Fix introducing invalid uses of intrinsics

[Matt Arsenault via Phabricator via llvm-commits]

arsenm created this revision.
arsenm added reviewers: regehr, aeubanks, fhahn, dblaikie, lebedev.ri.
Herald added a subscriber: StephenFan.
Herald added a project: All.
arsenm requested review of this revision.
Herald added a subscriber: wdng.
Herald added a project: LLVM.

https://reviews.llvm.org/D153313

Files:
  llvm/test/tools/llvm-reduce/operands-skip-intrinsics.ll
  llvm/tools/llvm-reduce/deltas/ReduceOperandsSkip.cpp


Index: llvm/tools/llvm-reduce/deltas/ReduceOperandsSkip.cpp
===================================================================
--- llvm/tools/llvm-reduce/deltas/ReduceOperandsSkip.cpp
+++ llvm/tools/llvm-reduce/deltas/ReduceOperandsSkip.cpp
@@ -163,6 +163,12 @@
         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
Index: llvm/test/tools/llvm-reduce/operands-skip-intrinsics.ll
===================================================================
--- /dev/null
+++ 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
+}


-------------- next part --------------
A non-text attachment was scrubbed...
Name: D153313.532771.patch
Type: text/x-patch
Size: 2673 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20230620/d6740b14/attachment.bin>