[PATCH] D13239: [ARM] Modify codegen for memcpy intrinsic to prefer LDM/STM.

scott douglass via llvm-commits llvm-commits at lists.llvm.org
Thu Oct 1 09:35:35 PDT 2015


scott-0 marked 2 inline comments as done.
scott-0 added a comment.

New version on it's way.


================
Comment at: lib/Target/ARM/InstPrinter/ARMInstPrinter.cpp:754
@@ +753,3 @@
+  std::copy(MI->begin() + OpNum, MI->end(), RegOps.begin());
+  std::sort(RegOps.begin(), RegOps.end(),
+            [this](const MCOperand &O1, const MCOperand &O2) -> bool {
----------------
jmolloy wrote:
> Use std::stable_sort instead of std::sort, for deterministicness.
Ok, but the registers being sorted are unique.


http://reviews.llvm.org/D13239





More information about the llvm-commits mailing list