[PATCH] D80062: [x86] Propagate memory operands during call frame optimization & ISel DAG postprocessing

[Craig Topper via Phabricator via llvm-commits]

craig.topper added a reviewer: rnk.
craig.topper added inline comments.


================
Comment at: llvm/lib/Target/X86/X86CallFrameOptimization.cpp:565
           Push->addOperand(DefMov->getOperand(i));
+        Push->setMemRefs(MF, DefMov->memoperands());
 
----------------
Kayjukh wrote:
> craig.topper wrote:
> > This captures the memoperand for the load of the source, but what about the store to the stack?
> I'm not really sure how to model this. As far as I can tell, we don't know the location to which the value gets stored on the stack. Would you add a memory operand with the MachinePointerInfo returned by MachinePointerInfo::getUnknownStack?
> Or am I missing something?
Does the Store node that we're replacing have a mem operand?


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D80062/new/

https://reviews.llvm.org/D80062