[PATCH] D57519: [X86][AVX] Add VMOVDDUP-VPBROADCASTQ execution domain mapping

Simon Pilgrim via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Jan 31 13:10:14 PST 2019


RKSimon marked an inline comment as done.
RKSimon added inline comments.


================
Comment at: test/CodeGen/X86/avx2-intrinsics-x86-upgrade.ll:242
+; AVX512:       ## %bb.0:
+; AVX512-NEXT:    vpbroadcastq %xmm0, %xmm0
+; AVX512-NEXT:    ret{{[l|q]}}
----------------
craig.topper wrote:
> Should we do this for VMOVDDUPZ128?
That was my question about AVX512 - it doesn't seem to have any broadcast domain entries so far. I'm happy to do the work but that might be worth a separate patch?


Repository:
  rL LLVM

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

https://reviews.llvm.org/D57519





More information about the llvm-commits mailing list