On Fri, 22 Sep 2023 23:38:49 GMT, Leonid Mesnik <lmes...@openjdk.org> wrote:
>> The code related to virtual threads support is moved into a separate class >> DebugeeWrapper. >> >> The code of method main() remains the same. I don't mix code change with >> moving code between files. >> >> Tesed by running >> make run-test JTREG_RETAIN=all TEST=com/sun/jdi >> make run-test JTREG_RETAIN=all JTREG_TEST_THREAD_FACTORY=Virtual >> TEST=com/sun/jdi >> locally and tier1 and hs-tier5 in Mach5 CI > > Leonid Mesnik has updated the pull request incrementally with one additional > commit since the last revision: > > fixed Looks good. Can you do a test run using TEST_VM_OPTS set to something? Maybe try with -Xcomp. Just want to make sure VM argument passing is working. ------------- Marked as reviewed by cjplummer (Reviewer). PR Review: https://git.openjdk.org/jdk/pull/15874#pullrequestreview-1640755933