[PATCH] D52085: [ARM][ARMLoadStoreOptimizer]

[Luke Cheeseman via Phabricator via llvm-commits]

LukeCheeseman added inline comments.


================
Comment at: lib/Target/ARM/ARMLoadStoreOptimizer.cpp:1054
+      if (Count > Limit) {
+        CanMergeToLSMulti = CanMergeToLSDouble = false;
+        break;
----------------
efriedma wrote:
> Instead of disabling merging, could we just do something like "if (Count == Limit) break;", so we merge the first 16 stores, then consider any remaining stores as a separate set?
Ah yes, that's a much better idea. Thanks


https://reviews.llvm.org/D52085