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

[Robbin Ehn]

On Wed, 2 Jun 2021 10:08:02 GMT, David Holmes <dholmes at openjdk.org> wrote:

> 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

Seems good!

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

Marked as reviewed by rehn (Reviewer).

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