Revert "[FunctionAttrs] Remove redundant assignment."

Tilmann Scheller · Tilmann Scheller · commit 73a482f4ccc1 · 2015-11-20T19:17:10.000Z
This reverts r253661. Turns out that the assignment is not redundant (despite the Clang static analyzer claiming the opposite). The variable is being used by the lambda function AddUsersToWorklistIfCapturing(). git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@253696 91177308-0d34-0410-b5e6-96231b3b80d8
diff --git a/lib/Transforms/IPO/FunctionAttrs.cpp b/lib/Transforms/IPO/FunctionAttrs.cpp
@@ -486,6 +486,8 @@ determinePointerReadAttrs(Argument *A,
         return Attribute::None;
       }
 
+      Captures &= !CS.doesNotCapture(UseIndex);
+
       // Since the optimizer (by design) cannot see the data flow corresponding
       // to a operand bundle use, these cannot participate in the optimistic SCC
       // analysis.  Instead, we model the operand bundle uses as arguments in

Original file line number	Diff line number	Diff line change
`@@ -486,6 +486,8 @@ determinePointerReadAttrs(Argument *A,`
`486`	`486`	`return Attribute::None;`
`487`	`487`	`}`
`488`	`488`
	`489`	`+ Captures &= !CS.doesNotCapture(UseIndex);`
	`490`	`+`
`489`	`491`	`// Since the optimizer (by design) cannot see the data flow corresponding`
`490`	`492`	`// to a operand bundle use, these cannot participate in the optimistic SCC`
`491`	`493`	`// analysis. Instead, we model the operand bundle uses as arguments in`