[PATCH] D26248: [AArch64] Removed the narrow load merging code in the ld/st optimizer.

Jun Bum Lim via llvm-commits llvm-commits at lists.llvm.org
Mon Nov 7 07:24:38 PST 2016


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

LGTM. Thanks Chad!



================
Comment at: lib/Target/AArch64/AArch64LoadStoreOptimizer.cpp:1450
     // us what the next instruction is after it's done mucking about.
     MBBI = mergeNarrowInsns(MBBI, MergeMI, Flags);
     return true;
----------------
Now, mergeNarrowInsns () is used only for narrow zero store merge. So, "mergeNarrowZeroStore" seems to be the right name.


https://reviews.llvm.org/D26248





More information about the llvm-commits mailing list