[PATCH] [CodeGen] Combine small-element shuffles of scalar_to_vector in terms of the wider scalar.

Ahmed Bougacha ahmed.bougacha at gmail.com
Wed Apr 8 09:52:18 PDT 2015


I'm not sure: I initially dismissed it because it works on multiple scalar inputs, for which it makes sense to do a BUILD_VECTOR directly.  For a single scalar input, I found it more straightforward to shuffle a SCALAR_TO_VECTOR, but I agree it's not very consistent.

Also, I'm not sure what happens to the BUILD_VECTOR later on, but I can imagine it being expanded to the shuffle anyway, so maybe it does make sense to do that instead.  Let me look into it!

-Ahmed


http://reviews.llvm.org/D8885

EMAIL PREFERENCES
  http://reviews.llvm.org/settings/panel/emailpreferences/






More information about the llvm-commits mailing list