[PATCH] [CodeGen] Combine small-element shuffles of scalar_to_vector in terms of the wider scalar.
Simon Pilgrim
llvm-dev at redking.me.uk
Wed Apr 8 11:18:41 PDT 2015
In http://reviews.llvm.org/D8885#153311, @ab wrote:
> 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.
If you look at the early versions of the patch on http://reviews.llvm.org/D8516 we did have a version that would create a SCALAR_TO_VECTOR if only the first scalar was defined, but there wasn't a good use case for it so it got dropped. Readding that path should be trivial.
http://reviews.llvm.org/D8885
EMAIL PREFERENCES
http://reviews.llvm.org/settings/panel/emailpreferences/
More information about the llvm-commits
mailing list