Integrated: 8271569: Clean up the use of CDS constants and field offsets
Yumin Qi
minqi at openjdk.java.net
Thu Sep 16 18:00:58 UTC 2021
On Thu, 9 Sep 2021 18:13:10 GMT, Yumin Qi <minqi at openjdk.org> wrote:
> Changed cdsOffsets.cpp to cdsConstants.cpp, now the offsets and constants are initialized static and searched separately.
> The offsets array could not use 'constexpr' since g++ on MacOs and VSC++ on Windows complained reinterpret_cast in 'offset_of' should not be used in constexpr initialization. Changed some field access for forming global list first.
>
> Note with 'git mv' to rename cdsoffset.cpp to cdsConstants.cpp, same for cdsoffsets.hpp to cdsConstants.hpp, due to the contents changed more than 50% so git will not think cdsConstants.cpp is renamed from cdsoffsets.cpp. Instead, it is regarded as a new file.
>
> Tests: ter1-4
>
> Thanks
> Yumin
This pull request has now been integrated.
Changeset: 9c5441c9
Author: Yumin Qi <minqi at openjdk.org>
URL: https://git.openjdk.java.net/jdk/commit/9c5441c9c4eea2ea4ba78ee426729bf646dc042d
Stats: 342 lines in 11 files changed: 170 ins; 126 del; 46 mod
8271569: Clean up the use of CDS constants and field offsets
Reviewed-by: iklam, dholmes
-------------
PR: https://git.openjdk.java.net/jdk/pull/5450
More information about the hotspot-runtime-dev
mailing list