The specification of JVMTI GetClassModifiers needs some TLC. The spec was 
carried over from the equivalent function in JVMDI but doesn't really specify 
that it works like Class::getModifiers and returns Java language modifiers and 
not necessarily the access flags from the class file. The change in behavior to 
only set ACC_IDENTITY when the class is an identity class or array makes it a 
good time to re-write the spec to align with existing and future behavior.

A new test is added to test this function with value classes. The tests are 
added to the "newer" serviceability/jvmti tree. There are some older tests for 
GetClassModifiers in vmTestbase/nsk suite so there is a bit of an overlap.

---------
- [x] I confirm that I make this contribution in accordance with the [OpenJDK 
Interim AI Policy](https://openjdk.org/legal/ai).

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

Commit messages:
 - Tweak wording
 - Initial commit

Changes: https://git.openjdk.org/valhalla/pull/2526/files
  Webrev: https://webrevs.openjdk.org/?repo=valhalla&pr=2526&range=00
  Issue: https://bugs.openjdk.org/browse/JDK-8386250
  Stats: 435 lines in 5 files changed: 421 ins; 1 del; 13 mod
  Patch: https://git.openjdk.org/valhalla/pull/2526.diff
  Fetch: git fetch https://git.openjdk.org/valhalla.git pull/2526/head:pull/2526

PR: https://git.openjdk.org/valhalla/pull/2526

Reply via email to