Changes look good to me,
Misha
On 05/09/2018 02:09 PM, Igor Ignatyev wrote:
http://cr.openjdk.java.net/~iignatyev//8199384/webrev.00/index.html
61414 lines changed: 61414 ins; 0 del; 0 mod;
Hi all,
could you please review this patch which open sources MLVM tests from VM
testbase?
these tests were developed in early days of JSR292 to test different aspects of
MethodHandles and invokedynamic.
As usually w/ VM testbase code, these tests are old, they have been run in
hotspot testing for a long period of time. Originally, these tests were run by
a test harness different from jtreg and had different build and execution
schemes, some parts couldn't be easily translated to jtreg, so tests might have
actions or pieces of code which look weird. In a long term, we are planning to
rework them.
JBS: https://bugs.openjdk.java.net/browse/JDK-8199384
webrev: http://cr.openjdk.java.net/~iignatyev//8199384/webrev.00/index.html
testing: :vmTestbase_vm_mlvm test group
Thanks,
-- Igor
