[PATCH] D14306: [CaptureTracking] Support operand bundles conservatively

Sanjoy Das via llvm-commits llvm-commits at lists.llvm.org
Wed Nov 4 15:23:38 PST 2015


This revision was automatically updated to reflect the committed changes.
Closed by commit rL252095: [CaptureTracking] Support operand bundles conservatively (authored by sanjoy).

Changed prior to commit:
  http://reviews.llvm.org/D14306?vs=39248&id=39270#toc

Repository:
  rL LLVM

http://reviews.llvm.org/D14306

Files:
  llvm/trunk/lib/Analysis/CaptureTracking.cpp
  llvm/trunk/test/Feature/OperandBundles/dse.ll

Index: llvm/trunk/test/Feature/OperandBundles/dse.ll
===================================================================
--- llvm/trunk/test/Feature/OperandBundles/dse.ll
+++ llvm/trunk/test/Feature/OperandBundles/dse.ll
@@ -0,0 +1,29 @@
+; RUN: opt -S -dse < %s | FileCheck %s
+
+declare void @f()
+declare noalias i8* @malloc(i32) nounwind
+
+define void @test_0() {
+; CHECK-LABEL: @test_0(
+  %m = call i8* @malloc(i32 24)
+  tail call void @f() [ "unknown"(i8* %m) ]
+; CHECK: store i8 -19, i8* %m
+  store i8 -19, i8* %m
+  ret void
+}
+
+define i8* @test_1() {
+; CHECK-LABEL: @test_1(
+  %m = call i8* @malloc(i32 24)
+  tail call void @f() [ "unknown"(i8* %m) ]
+  store i8 -19, i8* %m
+  tail call void @f()
+  store i8 101, i8* %m
+
+; CHECK: tail call void @f() [ "unknown"(i8* %m) ]
+; CHECK: store i8 -19, i8* %m
+; CHECK: tail call void @f()
+; CHECK: store i8 101, i8* %m
+
+  ret i8* %m
+}
Index: llvm/trunk/lib/Analysis/CaptureTracking.cpp
===================================================================
--- llvm/trunk/lib/Analysis/CaptureTracking.cpp
+++ llvm/trunk/lib/Analysis/CaptureTracking.cpp
@@ -253,8 +253,9 @@
       // that loading a value from a pointer does not cause the pointer to be
       // captured, even though the loaded value might be the pointer itself
       // (think of self-referential objects).
-      CallSite::arg_iterator B = CS.arg_begin(), E = CS.arg_end();
-      for (CallSite::arg_iterator A = B; A != E; ++A)
+      CallSite::data_operand_iterator B =
+        CS.data_operands_begin(), E = CS.data_operands_end();
+      for (CallSite::data_operand_iterator A = B; A != E; ++A)
         if (A->get() == V && !CS.doesNotCapture(A - B))
           // The parameter is not marked 'nocapture' - captured.
           if (Tracker->captured(U))


-------------- next part --------------
A non-text attachment was scrubbed...
Name: D14306.39270.patch
Type: text/x-patch
Size: 1798 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20151104/08669a49/attachment.bin>


More information about the llvm-commits mailing list