[PATCH] D39134: [X86][SSE] Add MOVHPSrm to domain tables

Craig Topper via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Oct 20 18:59:40 PDT 2017


craig.topper added inline comments.


================
Comment at: lib/Target/X86/X86InstrInfo.cpp:9446
   { X86::XORPSrr,    X86::XORPDrr,   X86::PXORrr    },
-  { X86::UNPCKLPDrm, X86::UNPCKLPDrm, X86::PUNPCKLQDQrm },
+  { X86::MOVHPSrm,   X86::UNPCKLPDrm, X86::PUNPCKLQDQrm },
   { X86::MOVLHPSrr,  X86::UNPCKLPDrr, X86::PUNPCKLQDQrr },
----------------
Does moving from MOVHPS to UNPCKLPD cause us to potentially access unaligned memory? MOVHPS doesn't have an alignment requirement, but UNPCKLPD does.


Repository:
  rL LLVM

https://reviews.llvm.org/D39134





More information about the llvm-commits mailing list