[lworld] RFR: 8317172: [lworld] vmTestbase/vm/mlvm/hiddenloader/stress/byteMutation/Test.java assert SystemDictionary::resolve_instance_class_or_null

Coleen Phillimore coleenp at openjdk.org
Tue Feb 24 14:50:31 UTC 2026


On Tue, 24 Feb 2026 14:16:38 GMT, Casper Norrbin <cnorrbin at openjdk.org> wrote:

> Hi everyone,
> 
> `ClassFileParser::verify_legal_class_name` has some leftover code referencing `CONSTANT_CLASS_DESCRIPTORS`, which is no longer used and relevant. With the code in place, we wrongly verifies classnames, resulting in test failures.
> 
> With this PR I have removed all references to `CONSTANT_CLASS_DESCRIPTORS`. The failing test now passes as expected and as such I have also removed it from the problemlist.
> 
> Testing:
> - Tiers 1-3

Looks good!

-------------

Marked as reviewed by coleenp (Committer).

PR Review: https://git.openjdk.org/valhalla/pull/2158#pullrequestreview-3848426591


More information about the valhalla-dev mailing list