On Tue, 2 Aug 2022 06:32:09 GMT, Ao Qi <a...@openjdk.org> wrote:

>> `vmTestbase/nsk/jvmti/GetThreadInfo/thrinfo001/TestDescription.java` and 
>> `vmTestbase/nsk/jvmti/RedefineClasses/StressRedefineVirtual/StressRedefineVirtual.java`
>>  are added or modified by 
>> [JDK-8284161](https://bugs.openjdk.org/browse/JDK-8284161), and they are 
>> failed if Loom or JVMTI is not supported.
>> 
>> The issue could be reproduced by zero.
>
> Ao Qi has updated the pull request with a new target base due to a merge or a 
> rebase. The incremental webrev excludes the unrelated changes brought in by 
> the merge/rebase. The pull request contains four additional commits since the 
> last revision:
> 
>  - Merge branch 'master' into 8291508
>  - XMLDecoder tests are filed in JDK-8291640
>  - XMLDecoder update
>    
>    Co-authored-by: Alan Bateman <al...@openjdk.org>
>  - 8291508: Fix some tests with "requires vm.continuations"

I'm surprised these two tests have the `--enable-preview` flag.
@lmesnik Could you, please, take a look at these two tests?
Otherwise, the fix looks good to me.
Thanks,
Serguei

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

Marked as reviewed by sspitsyn (Reviewer).

PR: https://git.openjdk.org/jdk/pull/9677

Reply via email to