[PATCH] D62726: [X86] Use fresh MemOps when emitting VAARG64

Simon Pilgrim via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Jun 13 02:11:11 PDT 2019


RKSimon added inline comments.


================
Comment at: llvm/lib/Target/X86/X86ISelLowering.cpp:28689-28690
+      OldMMO->getRanges(), OldMMO->getSyncScopeID(), OldMMO->getOrdering(),
+      OldMMO->getFailureOrdering());
 
   // Machine Information
----------------
Would it be worth adding a new getMachineMemOperand version that just updates the flags?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D62726





More information about the llvm-commits mailing list