[lworld] Integrated: 8329108: [lworld] valhalla/valuetypes/ObjectMethods.java fails with default CDS archive
Matias Saavedra Silva
matsaave at openjdk.org
Thu Jul 25 17:39:00 UTC 2024
On Wed, 24 Jul 2024 21:14:14 GMT, Matias Saavedra Silva <matsaave at openjdk.org> wrote:
> When this test was run after `java -Xshare:dump --enable-preview`, the test would crash due to a null signature calling convention. This occurred because `has_scalarized_args`, a flag whose value should be determined at runtime when regenerating the method adapter, was stored in ConstMethod and then archived. The resulting mismatch in a subsequent run meant the wrong `sig_cc` would be used. This change moves the flags in question into `method.hpp` and avoids dumping them so they can be properly regenerated at runtime. The test was verified locally and the changes were further verified with tier 1-5 tests.
This pull request has now been integrated.
Changeset: 9f3f535b
Author: Matias Saavedra Silva <matsaave at openjdk.org>
Committer: Frederic Parain <fparain at openjdk.org>
URL: https://git.openjdk.org/valhalla/commit/9f3f535b72857453c02a7c0af1ef9415dd1048bb
Stats: 19 lines in 7 files changed: 6 ins; 11 del; 2 mod
8329108: [lworld] valhalla/valuetypes/ObjectMethods.java fails with default CDS archive
Reviewed-by: dsimms, fparain
-------------
PR: https://git.openjdk.org/valhalla/pull/1180
More information about the valhalla-dev
mailing list