[PATCH] ARM: Implement big endian bit-conversion for NEON types

James Molloy james.molloy at arm.com
Fri May 9 06:56:20 PDT 2014


Hi Christian,

Sorry, must have missed your comment first time round.

> In hard-mode vectors are passed using the vector registers. In such a case these conversions are triggered by BITCAST operations only.

Indeed - this is what I was getting at. So where are those BITCASTs inserted in your patch? They aren't, and I don't think your patch will work in hard float mode. So do you want to make a followon patch to work for hard-float mode, or do you want me to treat this patch as working for hard-float mode as well (as you mentioned in an earlier comment)?

Cheers,
James

http://reviews.llvm.org/D3651






More information about the llvm-commits mailing list