[PATCH] D52085: [ARM][ARMLoadStoreOptimizer]

[Eli Friedman via Phabricator via llvm-commits]

efriedma added inline comments.


================
Comment at: lib/Target/ARM/ARMLoadStoreOptimizer.cpp:1054
+      if (Count > Limit) {
+        CanMergeToLSMulti = CanMergeToLSDouble = false;
+        break;
----------------
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?


https://reviews.llvm.org/D52085