[llvm] r253664 - [Hexagon] Remove redundant assignment.

[David Blaikie via llvm-commits]

Not sure if that's the right fix - it's posisble that the later use of
"Changed" should've been "Changed |=" instead of "Changed =" - again,
probably best to address this by replying to the original commit, rather
than just making the change outright like this.

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

> Author: tilmann
> Date: Fri Nov 20 07:27:30 2015
> New Revision: 253664
>
> URL: http://llvm.org/viewvc/llvm-project?rev=253664&view=rev
> Log:
> [Hexagon] Remove redundant assignment.
>
> Identified by the Clang static analyzer.
>
> Modified:
>     llvm/trunk/lib/Target/Hexagon/HexagonGenInsert.cpp
>
> Modified: llvm/trunk/lib/Target/Hexagon/HexagonGenInsert.cpp
> URL:
> http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/Target/Hexagon/HexagonGenInsert.cpp?rev=253664&r1=253663&r2=253664&view=diff
>
> ==============================================================================
> --- llvm/trunk/lib/Target/Hexagon/HexagonGenInsert.cpp (original)
> +++ llvm/trunk/lib/Target/Hexagon/HexagonGenInsert.cpp Fri Nov 20 07:27:30
> 2015
> @@ -1495,7 +1495,7 @@ bool HexagonGenInsert::runOnMachineFunct
>    // version of DCE that preserves lifetime markers. Without it, merging
>    // of stack objects can fail to recognize and merge disjoint objects
>    // leading to unnecessary stack growth.
> -  Changed |= removeDeadCode(MDT->getRootNode());
> +  removeDeadCode(MDT->getRootNode());
>
>    const HexagonEvaluator HE(*HRI, *MRI, *HII, MF);
>    BitTracker BTLoc(HE, MF);
>
>
> _______________________________________________
> 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/35a81768/attachment.html>