[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