[PATCH] D76508: [VectorUtils] move x86's scaleShuffleMask to generic VectorUtils

Sanjay Patel via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Sun Mar 22 09:04:09 PDT 2020


spatel added a comment.

In D76508#1934379 <https://reviews.llvm.org/D76508#1934379>, @RKSimon wrote:

> This shouldn't be endian specific - DAGCombiner::visitVECTOR_SHUFFLE already does SHUFFLE(BITCAST,UNDEF)->BITCAST(SHUFFLE) with its own almost-identical version of ScaleShuffleMask.


Thanks - I knew that we had this somewhere else, but I missed that. I can remove that duplication as part of this patch if that improves the motivation.


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

https://reviews.llvm.org/D76508





More information about the llvm-commits mailing list