[PATCH] D57156: [x86] narrow a shuffle that doesn't use or set any high elements

Sanjay Patel via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Jan 24 16:36:39 PST 2019


spatel added a comment.

Note: Tests with AVX512 and 512-bit vectors are not showing the same improvements as other targets/tests because we don't know how to collapse an (extract (extract V, 0), 0) into 1 extract node yet. I'm imagining that patch is similar to the insert-of-insert that in D56604 <https://reviews.llvm.org/D56604>. Once we have that fold, we'll get several more narrowing of binops like we see here. But like this patch, that one also appears to expose a shuffle lowering hole, so it might need a preliminary patch to avoid a regression.


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

https://reviews.llvm.org/D57156





More information about the llvm-commits mailing list