On Tue, 19 May 2026 12:00:14 GMT, Coleen Phillimore <[email protected]> wrote:
>> This change makes UseCompactObjectHeaders default and switches the sense of >> the CDS archive naming. Now classes.jsa is with UseCompressedOops + >> UseCompactObjectHeaders (both default). Then theres: >> classes_nocoops.jsa >> classes_nocoh.jsa >> classes_nocoops_nocoh.jsa >> >> Hopefully someday we won't need the nocoh versions. >> >> Tested with tier1-8. With only one issue also fixed in this change, and no >> other new issues found. >> >> - [x] I confirm that I make this contribution in accordance with the >> [OpenJDK Interim AI Policy](https://openjdk.org/legal/ai). > > Coleen Phillimore has updated the pull request with a new target base due to > a merge or a rebase. The pull request now contains 10 commits: > > - Merge branch 'master' into coh-default > - Fix comment. > - Fix CDSPlugin and CDSPluginTest, and VMOptionWarning.java test also. > - Fix comments for nocoh in Images.gmk. > - Merge branch 'master' into coh-default > - Fix CDSPluginTest and remove comment. > - Reworked OMT/COH warning to always warn with -OMT and +COH. Fixed build > --enable-cds-archive-coh to nocoh. > - Update test/hotspot/jtreg/runtime/CommandLine/VMOptionWarning.java > > Co-authored-by: Fredrik Bredberg <[email protected]> > - Add warning about incompatible COH with OMT. > - Make UseCompactObjectHeaders the default and adjust CDS archives. It still looks good to me. ------------- Marked as reviewed by fbredberg (Reviewer). PR Review: https://git.openjdk.org/jdk/pull/30322#pullrequestreview-4319209266
