On 25/08/2017 00:36, mandy chung wrote:
:

That's a good idea.  I added a new test in hotspot/test/compiler/jvmci.  Updated webrev: http://cr.openjdk.java.net/~mchung/jdk10/webrevs/8186145/webrev.01/index.html

I considered adding this in the jdk/test directory.  @requires vm.jvmci tightly couples with VM and also VM testlibrary and I am hesitated extending jdk/test/TEST.ROOT to set up for this test. We can revisit this if there is a strong need to move this new test in the jdk/test directory.

The updated webrev looks good.

-Alan

Reply via email to