[llvm] r253661 - [FunctionAttrs] Remove redundant assignment.

David Blaikie via llvm-commits llvm-commits at lists.llvm.org
Fri Nov 20 10:17:01 PST 2015


Might want to follow up with wherever this was committed (or wherever the
store became dead) to check that there wasn't an intended use of this value
that just went missing/was never finished...

On Fri, Nov 20, 2015 at 4:51 AM, Tilmann Scheller via llvm-commits <
llvm-commits at lists.llvm.org> wrote:

> Author: tilmann
> Date: Fri Nov 20 06:51:58 2015
> New Revision: 253661
>
> URL: http://llvm.org/viewvc/llvm-project?rev=253661&view=rev
> Log:
> [FunctionAttrs] Remove redundant assignment.
>
> Identified by the Clang static analyzer.
>
> Modified:
>     llvm/trunk/lib/Transforms/IPO/FunctionAttrs.cpp
>
> Modified: llvm/trunk/lib/Transforms/IPO/FunctionAttrs.cpp
> URL:
> http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/Transforms/IPO/FunctionAttrs.cpp?rev=253661&r1=253660&r2=253661&view=diff
>
> ==============================================================================
> --- llvm/trunk/lib/Transforms/IPO/FunctionAttrs.cpp (original)
> +++ llvm/trunk/lib/Transforms/IPO/FunctionAttrs.cpp Fri Nov 20 06:51:58
> 2015
> @@ -486,8 +486,6 @@ 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
>
>
> _______________________________________________
> llvm-commits mailing list
> llvm-commits at lists.llvm.org
> http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-commits
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20151120/551f0efa/attachment.html>


More information about the llvm-commits mailing list