[PATCH] D122216: [X86] Rename MMX_MOVD64from64rm to MMX_MOVD64from64mr b/c it stores sth, NFC

Simon Pilgrim via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Mar 22 02:57:07 PDT 2022


RKSimon accepted this revision.
RKSimon added a comment.

LGTM - cheers



================
Comment at: llvm/lib/Target/X86/X86SchedSandyBridge.td:819
 }
-def: InstRW<[SBWriteResGroup48], (instrs MMX_MOVD64from64rm,
+def: InstRW<[SBWriteResGroup48], (instrs MMX_MOVD64from64mr,
                                          VBROADCASTSSrm)>;
----------------
The fact that this is a store makes me suspicious that it shouldn't be with all these loads - I'll take a look at it.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D122216



More information about the llvm-commits mailing list