RFR: 8174957: [JVMCI] jaotc is broken in Xcomp mode

[Igor Veresov]

Looks good, but I think the following bit would also be required to fix this particular issue:

diff --git a/src/jdk.aot/share/classes/jdk.tools.jaotc/src/jdk/tools/jaotc/DataBuilder.java b/src/jdk.aot/share/classes/jdk.tools.jaotc/src/jdk/tools/jaotc/DataBuilder.java
--- a/src/jdk.aot/share/classes/jdk.tools.jaotc/src/jdk/tools/jaotc/DataBuilder.java
+++ b/src/jdk.aot/share/classes/jdk.tools.jaotc/src/jdk/tools/jaotc/DataBuilder.java
@@ -70,7 +70,7 @@
      */
     private void fillVMAddresses(HotSpotVMConfigStore config) {
         for (VMField vmField : config.getFields().values()) {
-            if (vmField.value != null) {
+            if (vmField.value != null && vmField.value instanceof Long) {
                 final long address = vmField.value;
                 String value = vmField.name;
                 /*

Thanks,
igor

> On Feb 15, 2017, at 2:24 AM, Doug Simon <doug.simon at oracle.com> wrote:
> 
> Please review this fix for a bug introduced by JDK-8173912. The value written by readConfiguation (in jvmciCompilerToVM.cpp) to VMField.value may now be a Boolean. As such, the type of VMField.value must be Object.
> 
> https://bugs.openjdk.java.net/browse/JDK-8174957
> http://cr.openjdk.java.net/~dnsimon/8174957/webrev/
> 
> -Doug

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.openjdk.java.net/pipermail/hotspot-compiler-dev/attachments/20170215/63e531d0/attachment-0001.html>