RFR: JDK-8302989: Add missing INCLUDE_CDS checks [v4]
Matthias Baesken
mbaesken at openjdk.org
Tue Mar 14 09:27:39 UTC 2023
On Fri, 3 Mar 2023 08:55:07 GMT, Matthias Baesken <mbaesken at openjdk.org> wrote:
>> The cds only coding in hotspot is usually guarded with the INCLUDE_CDS macro so that it can be removed at compile time in case the correct configure flags are set.
>> However at some places INCLUDE_CDS is missing and should be added.
>>
>> One question - should (additionally to the UseSharedSpaces code section) the DumpSharedSpaces code sections be guarded as well with INCLUDE_CDS macros ?
>
> Matthias Baesken has updated the pull request incrementally with one additional commit since the last revision:
>
> Adjust arguments handling
I noticed that the Xshare-handling IS already an issue on AIX - there cds is not supported and `-Xshare:off` does not work.
So tests like the ones I mentioned in my list above e.g.
java/math/BigInteger/largeMemory/DivisionOverflow.java
all fail on AIX (like on other platforms without cds).
-------------
PR: https://git.openjdk.org/jdk/pull/12691
More information about the hotspot-dev
mailing list