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

Justin King jcking at openjdk.org
Tue Jan 24 16:02:26 UTC 2023


On Tue, 24 Jan 2023 07:15:48 GMT, David Holmes <dholmes at openjdk.org> wrote:

> Looking like a good cleanup but can we not put the `swap_uN` definitions in `byteswap.hpp` so we don't repeat then for each platform? Thanks.

Okay, looks like there is a single user of `Bytes::swap_uN` outside of Bytes, some code in JFR. I directly updated them to use `byteswap` and removed `Bytes::swap_uN` all together.

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

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


More information about the hotspot-dev mailing list