[llvm] [X86InstrInfo] support memfold on spillable inline asm (PR #70832)

Nick Desaulniers via llvm-commits llvm-commits at lists.llvm.org
Mon Nov 20 14:03:45 PST 2023


================
@@ -10333,5 +10333,15 @@ void X86InstrInfo::genAlternativeCodeSequence(
   }
 }
 
+// See also: X86DAGToDAGISel::SelectInlineAsmMemoryOperand().
+void X86InstrInfo::getFrameIndexOperands(SmallVectorImpl<MachineOperand> &Ops) const {
+  Ops.append({
+    MachineOperand::CreateImm(1),        // Scale
+    MachineOperand::CreateReg(0, false), // Index
+    MachineOperand::CreateImm(0),        // Disp
+    MachineOperand::CreateReg(0, false), // Segment
+  });
----------------
nickdesaulniers wrote:

Sorry for the excessive delays following up on this; have been traveling for conference season for the past two weeks.  I've just not made the changes you've suggested, and I think I do indeed like it better.  Will post a commit on top once my tests finish successfully locally.

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


More information about the llvm-commits mailing list