RFR: JDK-8300783: Consolidate byteswap implementations [v8]

David Holmes dholmes at openjdk.org
Wed Jan 25 01:33:05 UTC 2023


On Tue, 24 Jan 2023 16:14:22 GMT, Justin King <jcking at openjdk.org> wrote:

>> Deduplicate byte swapping implementations by consolidating them into `utilities/byteswap.hpp`, following `std::byteswap` introduced in C++23. Further simplification of `Bytes` will follow in https://github.com/openjdk/jdk/pull/12078.
>
> Justin King has updated the pull request incrementally with one additional commit since the last revision:
> 
>   Update comment
>   
>   Signed-off-by: Justin King <jcking at google.com>

Cleanup/consolidation looks really good. Have to wonder if we can go further with `get_Java_u2` etc, in a next step?

The actual byteswap stuff seems okay but not really my area and compiler specific actions are not something I'm at all familiar with.

I will grab this and run through our CI. Thanks.

-------------

PR: https://git.openjdk.org/jdk/pull/12114


More information about the hotspot-jfr-dev mailing list