[PATCH] [X86] Make EltsFromConsecutiveLoads code path work only on vectors it expects

Sanjay Patel spatel at rotateright.com
Tue Dec 9 13:34:18 PST 2014


>>! In D6579#5, @mkuper wrote: 
> Regarding vector loads of i8 / i16 - I believe that didn't work to begin with.
> The existing code fires only when NumElems == 4, and creates a ResNode of type v2i64. So for the bitcast in the end to be valid, the element size must be 32 bit. Or did I miss something?

Ah, ok. Please add a 'TODO' comment there that we should handle other types and/or file a PR. I don't think there's any reason to limit the optimization to just 32-bit elements.

http://reviews.llvm.org/D6579






More information about the llvm-commits mailing list