[PATCH] D147507: [X86] Disable masked UNPCKLPD/UNPCKHPD -> SHUFPS transformation

Noah Goldstein via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Apr 4 07:59:12 PDT 2023


goldstein.w.n added a comment.

In D147507#4242658 <https://reviews.llvm.org/D147507#4242658>, @RKSimon wrote:

> @goldstein.w.n You might want to investigate if its worth using VSHUFPD instead?

@pengfei and @RKSimon what about using `{VP}UNPCK{L|H}QDQ{...}`? I tested on ICL and didn't see any domain penalty. Wasn't able to find the hardware to test hsw/skl/.... and not sure if it falls under no-shuffle `hasNoDomainDelayShuffle` or something else but it is the ideal replacement both from perf and codesize perspective.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D147507



More information about the llvm-commits mailing list