[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