RFR: JDK-8300783: Consolidate byteswap implementations [v14]
Justin King
jcking at openjdk.org
Wed Feb 15 15:39:14 UTC 2023
> 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 with a new target base due to a merge or a rebase. The incremental webrev excludes the unrelated changes brought in by the merge/rebase. The pull request contains 24 additional commits since the last revision:
- Merge remote-tracking branch 'upstream/master' into byteswap
- Update based on review
Signed-off-by: Justin King <jcking at google.com>
- Fix copyright
Signed-off-by: Justin King <jcking at google.com>
- Update copyright
Signed-off-by: Justin King <jcking at google.com>
- Add missing include
Signed-off-by: Justin King <jcking at google.com>
- Remove unused include
Signed-off-by: Justin King <jcking at google.com>
- Reorganize tests
Signed-off-by: Justin King <jcking at google.com>
- Fix test
Signed-off-by: Justin King <jcking at google.com>
- Merge remote-tracking branch 'upstream/master' into byteswap
- Be restrict on requiring 1, 2, 4, or 8 byte integers
Signed-off-by: Justin King <jcking at google.com>
- ... and 14 more: https://git.openjdk.org/jdk/compare/8697eb93...223d733b
-------------
Changes:
- all: https://git.openjdk.org/jdk/pull/12114/files
- new: https://git.openjdk.org/jdk/pull/12114/files/f2ea0038..223d733b
Webrevs:
- full: https://webrevs.openjdk.org/?repo=jdk&pr=12114&range=13
- incr: https://webrevs.openjdk.org/?repo=jdk&pr=12114&range=12-13
Stats: 78274 lines in 2100 files changed: 30636 ins; 20323 del; 27315 mod
Patch: https://git.openjdk.org/jdk/pull/12114.diff
Fetch: git fetch https://git.openjdk.org/jdk pull/12114/head:pull/12114
PR: https://git.openjdk.org/jdk/pull/12114
More information about the hotspot-jfr-dev
mailing list