[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