[PATCH] D14489: [AArch64] Applying load pair optimization for volatile load/store

Chad Rosier via llvm-commits llvm-commits at lists.llvm.org
Fri Nov 13 09:53:40 PST 2015


mcrosier added a comment.

I also wanted to point out that this patch is awesome in that it also allows instructions whos MMOs have been dropped to be merged (hasOrderedMemoryRef makes a conservative assumption when MMOs are missing).  Therefore, this solution will transform more that just volatile loads/stores.  Of course the correct fix is to probably not drop MMO (as is don't in tail merging)...


http://reviews.llvm.org/D14489





More information about the llvm-commits mailing list