[PATCH] D12775: Use new TokenFactor chain when merging stores

Mikael Holmén via llvm-commits llvm-commits at lists.llvm.org
Wed Sep 16 06:33:28 PDT 2015


I've been running with this patch for a while now and it solves the 
original problem I had and no new issues have come up so far.

/Mikael

On 09/12/2015 06:50 AM, Matt Arsenault via llvm-commits wrote:
> arsenm updated this revision to Diff 34619.
> arsenm added a comment.
>
> Fix not adding 1st store's chain to new TokenFactor
>
>
> http://reviews.llvm.org/D12775
>
> Files:
>    lib/CodeGen/SelectionDAG/DAGCombiner.cpp
>    test/CodeGen/X86/2012-11-28-merge-store-alias.ll
>    test/CodeGen/X86/merge-store-partially-alias-loads.ll
>
>
>
> _______________________________________________
> llvm-commits mailing list
> llvm-commits at lists.llvm.org
> http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-commits
>


More information about the llvm-commits mailing list