erichkeane added a comment.

In https://reviews.llvm.org/D37448#861443, @efriedma wrote:

> > According to our definition, v4si is NOT a "Vector" type, since a vector 
> > type requires it be a FP value.
>
> Umm, what?  An integer vector is still a vector.  The backend will return it 
> in xmm0 on x86 (both 32 and 64-bit).


Ugg.. you're right.  I missed that addReturnRegisterOutputs was creating a 
return type as an integer, so when it was being checked in this if/else tree 
(edited above) you're not actually looking at the return type anymore....


Repository:
  rL LLVM

https://reviews.llvm.org/D37448



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to