RFR (S) 8238289: Use _byteswap_ functions to implement Bytes::swap_uX on Windows

Schmelter, Ralf ralf.schmelter at sap.com
Fri Apr 3 08:40:57 UTC 2020


can you please review this change. It uses the VS compiler intrinsics to implement byte order swapping for 16, 32 and 64 bit types. For most other platforms we already used the platform specific calls to do this.

bugreport: https://bugs.openjdk.java.net/browse/JDK-8238289 
webrev: http://cr.openjdk.java.net/~rschmelter/webrevs/8238289/webrev.0/ 

As a consequence of this change the size of the server.dll increased by ~ 7kB for a release build. This was not expected, since the code needed for a implementation of the byte swap calls itself was much smaller now. In the end I've traced it down to more inlining being done in the JFR code.

Best regards,

More information about the hotspot-runtime-dev mailing list