llvm-reduce: Fix introducing invalid uses of intrinsics

[aeubanks]

lg

[arsenm]

29ce3678c0188a4c9ee6576934c3e4abd72ff12d

llvm/test/tools/llvm-reduce/operands-skip-intrinsics.ll

+; 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
+}

llvm/tools/llvm-reduce/deltas/ReduceOperandsSkip.cpp

for (Use &Op : I.operands()) {
       std::vector<Value *> Candidates = ReferencedVals.takeVector();
 
       // Remove ineligible candidates.
       llvm::erase_if(Candidates, [&, OpVal](Value *V) {
         // Candidate value must have the same type.
         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
         // pass eventually reach a fixpoint without switch back and forth
         // between two opportunities with the same reductive power.
         return !IsMoreReduced(V, OpVal);
       });