[PATCH] D55108: [AArch64] Re-run load/store optimizer after aggressive tail duplication

Dave Green via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Dec 13 10:09:58 PST 2018


dmgreen accepted this revision.
dmgreen added a comment.
This revision is now accepted and ready to land.

This looks find to me then, especially at aggressive opt level. IIRC, there is a test for the pass pipeline I would expect needs updating.

Other than that, LGTM



================
Comment at: test/CodeGen/AArch64/ldst-opt-after-block-placement.ll:31
   %1 = load i64, i64* %0, align 8
   %2 = getelementptr inbounds i64, i64* %ptr, i64 3
   store i64 0, i64* %0, align 8
----------------
%2 is never used here.


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D55108/new/

https://reviews.llvm.org/D55108





More information about the llvm-commits mailing list