[PATCH] D14936: Fix fptosi, fptoui from f16 vectors to i8, i16 vectors

James Molloy via llvm-commits llvm-commits at lists.llvm.org
Tue Dec 8 01:49:15 PST 2015


jmolloy requested changes to this revision.
jmolloy added a comment.
This revision now requires changes to proceed.

Hi Pirama,

Why are we only doing this for v4f16 and v8f16? What about oversized vectors - is there any specific reason you're excluding these? I'm not even sure oversized/undersized vectors can get this far, but it seems strange to whitelist.

I also think this code should be sunk down into LowerVectorFP_TO_INT.

Cheers,

James


http://reviews.llvm.org/D14936





More information about the llvm-commits mailing list