Skip to content

Make UseCompactObjectHeaders the default and adjust CDS archives.#2246

Draft
coleenp wants to merge 1 commit intoopenjdk:lworldfrom
coleenp:coh-default
Draft

Make UseCompactObjectHeaders the default and adjust CDS archives.#2246
coleenp wants to merge 1 commit intoopenjdk:lworldfrom
coleenp:coh-default

Conversation

@coleenp
Copy link
Contributor

@coleenp coleenp commented Mar 19, 2026

This change makes UseCompactObjectHeaders the default much like openjdk/jdk#30322

The archives here are for all the combinations.

classes_nocoh_preview.jsa
classes_nocoops_preview.jsa
classes_nocoh.jsa
classes_nocoops.jsa
classes_nocoops_nocoh_preview.jsa
classes_preview.jsa
classes_nocoops_nocoh.jsa
classes.jsa

Tested with tier1-8. Still working through the failed tests. Some are test errors. Will link them together.


Progress

  • Change must not contain extraneous whitespace
  • Commit message must refer to an issue
  • Change must be properly reviewed (1 review required, with at least 1 Committer)

Reviewing

Using git

Checkout this PR locally:
$ git fetch https://git.openjdk.org/valhalla.git pull/2246/head:pull/2246
$ git checkout pull/2246

Update a local copy of the PR:
$ git checkout pull/2246
$ git pull https://git.openjdk.org/valhalla.git pull/2246/head

Using Skara CLI tools

Checkout this PR locally:
$ git pr checkout 2246

View PR using the GUI difftool:
$ git pr show -t 2246

Using diff file

Download this PR as a diff file:
https://git.openjdk.org/valhalla/pull/2246.diff

@bridgekeeper
Copy link

bridgekeeper bot commented Mar 19, 2026

👋 Welcome back coleenp! A progress list of the required criteria for merging this PR into lworld will be added to the body of your pull request. There are additional pull request commands available for use with this pull request.

@openjdk
Copy link

openjdk bot commented Mar 19, 2026

❗ This change is not yet ready to be integrated.
See the Progress checklist in the description for automated requirements.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

1 participant