[PATCH] D80500: [X86] Add pseudo instructions to use MULX with a single destination when the low result isn't used.

Simon Pilgrim via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Sat May 30 02:37:01 PDT 2020


RKSimon accepted this revision.
RKSimon added a comment.
This revision is now accepted and ready to land.

LGTM with one trivial minor



================
Comment at: llvm/lib/Target/X86/X86MCInstLower.cpp:2218
+
+    for (unsigned I = 1; I < MI->getNumOperands(); ++I)
+      if (auto Op = MCInstLowering.LowerMachineOperand(MI, MI->getOperand(I)))
----------------
for (unsigned I = 1, E =  MI->getNumOperands(); I < E; ++I) ??


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D80500





More information about the llvm-commits mailing list