Forgot to tell that a copyright comment needs a year update.
No need in new webrev.
Thanks,
Serguei
On 7/24/18 17:48, serguei.spit...@oracle.com wrote:
Hi Katya,
Nice simple fix.
Thank you for taking care about it!
Thanks,
Serguei
On 7/24/18 15:10, Ekaterina Pavlova wrote:
Hi All,
serviceability/jvmti/GetModulesInfo/JvmtiGetAllModulesTest.java fails
with Graal because
two more modules jdk.proxy1 and jdk.proxy2 are dynamically
initialized by Graal code.
These modules are not part of boot modules and as results the check
fails.
It was agreed with Serviceability team to filter these modules out.
Please review the fix.
JBS: https://bugs.openjdk.java.net/browse/JDK-8195156
webrev:
http://cr.openjdk.java.net/~epavlova//8195156/webrev.00/index.html
testing: tested by running
serviceability/jvmti/GetModulesInfo/JvmtiGetAllModulesTest.java with
Graal and -Xcomp
Thanks,
-katya
p.s.
Igor Ignatyev volunteered to sponsor this change.