[llvm] [AArch64] New subtarget features to control ldp and stp formation, fo… (PR #66098)

David Green via llvm-commits llvm-commits at lists.llvm.org
Tue Sep 12 14:21:23 PDT 2023


================
@@ -2159,6 +2171,24 @@ bool AArch64LoadStoreOpt::tryToPairLdStInst(MachineBasicBlock::iterator &MBBI) {
     // Keeping the iterator straight is a pain, so we let the merge routine tell
     // us what the next instruction is after it's done mucking about.
     auto Prev = std::prev(MBBI);
+
+    // Get the needed alignments to check them if
+    // ldp-aligned-only/stp-aligned-only features are opted.
+    uint64_t memAlignment = memOp->getAlign().value();
----------------
davemgreen wrote:

LLVM uses CamelCase for variable names. So MemAlignment and TypeAlignment, and MemOp up above.

https://github.com/llvm/llvm-project/pull/66098


More information about the llvm-commits mailing list