RFR: 8268094: Some vmTestbase/nsk tests fail after ACC_STRICT/strictfp changes

David Holmes dholmes at openjdk.java.net
Wed Jun 2 10:32:41 UTC 2021


Part of the JEP-306 integration wasn't tested past tier 3 and missed some changes needed in the vmTestbase/nsk tests. The crux of the changes for JEP-306 are that the strictfp modifier has no affect any more (all fp math is strict) and is not observable as a modifier. At the VM level ACC_STRICT is no longer defined for classfiles version 61+ (ie JDK 17+).

Affected tests:

- vmTestbase/nsk/jvmti/GetMethodModifiers/methmod001/

This test has a part that expects to read the ACC_STRICT modifier for a strictfp method - so that part is just deleted.

Other jvmti test files refer to ACC_STRICT (but don't test anything) and so those references are also deleted.

- vmTestbase/nsk/jdi/VirtualMachine/redefineClasses/redefineclasses007.java
- vmTestbase/nsk/jdi/VirtualMachine/redefineClasses/redefineclasses008.java

These each have two subtests (07 and 08) that test changing the strictfp modifier on a method. These subtests are just deleted.

Testing:
 - local testing of affected test areas
 - tiers 1-5 also submitted for good measure (but may take a while to run and should not delay getting these fixes in places so that the CI returns to normal - thanks).

Thanks,
David

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

Commit messages:
 - 8268094: Some vmTestbase/nsk tests fail after ACC_STRICT/strictfp changes

Changes: https://git.openjdk.java.net/jdk/pull/4302/files
 Webrev: https://webrevs.openjdk.java.net/?repo=jdk&pr=4302&range=00
  Issue: https://bugs.openjdk.java.net/browse/JDK-8268094
  Stats: 339 lines in 14 files changed: 0 ins; 326 del; 13 mod
  Patch: https://git.openjdk.java.net/jdk/pull/4302.diff
  Fetch: git fetch https://git.openjdk.java.net/jdk pull/4302/head:pull/4302

PR: https://git.openjdk.java.net/jdk/pull/4302


More information about the serviceability-dev mailing list