[PATCH] D33675: [DAG] Do MergeConsecutiveStores again before Instruction Selection
Nirav Dave via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Mon Nov 13 07:06:04 PST 2017
niravd added a comment.
If the Aarch64 tests look good, I think this should be landable. The only remaining test that is more than a simple merge is the Mips/cconv/vector.ll test where the store merging allows the value stored on the stack to be forwarded to a now matching load and the stores and loads excised.
In https://reviews.llvm.org/D33675#918302, @efriedma wrote:
> The AArch64 test changes look fine.
https://reviews.llvm.org/D33675
More information about the llvm-commits
mailing list