VectorAPI Constant Shift for Byte and Short Intrinsic Support

Viswanathan, Sandhya sandhya.viswanathan at intel.com
Thu Jan 10 21:29:29 UTC 2019


Hi Shravya,

Could the following line in library_call.cpp be simplified using expression like (type2aelembytes(bt) * BitsPerByte - 1)?

7462   juint mask = (bt == T_LONG) ? (BitsPerLong - 1) : (bt == T_BYTE) ? (BitsPerByte - 1) : (bt == T_SHORT) ? (BitsPerShort - 1) : (BitsPerInt - 1);

Rest looks fine.

Thanks,
Sandhya

-----Original Message-----
From: panama-dev [mailto:panama-dev-bounces at openjdk.java.net] On Behalf Of Rukmannagari, Shravya
Sent: Thursday, January 10, 2019 10:36 AM
To: panama-dev <panama-dev at openjdk.java.net>
Subject: VectorAPI Constant Shift for Byte and Short Intrinsic Support

Hi All,

I would like to contribute a patch to add constant shifts support in Java for byte and short datatypes.



Could you please review the patch here:

http://cr.openjdk.java.net/~srukmannagar/VectorAPI_ByteShortConstantShift/webrev.00/



Thanks,

Shravya.




More information about the panama-dev mailing list