Re: RFR (XXL): JEP 243: Java-Level JVM Compiler Interface

2015-09-18 Thread Volker Simonis
For the top-level change I always get a strange error when importing it: patch failed, unable to continue (try -v) patch failed, rejects left in working dir errors during apply, please fix and refresh 8062493_jvmci_top_0911.v1.patch It is because of the strange path syntax of the last hunk in

Re: RFR (XXL): JEP 243: Java-Level JVM Compiler Interface

2015-09-18 Thread Christian Thalinger
> On Sep 17, 2015, at 10:20 PM, Volker Simonis wrote: > > For the top-level change I always get a strange error when importing it: > > patch failed, unable to continue (try -v) > patch failed, rejects left in working dir > errors during apply, please fix and refresh

Re: RFR (XXL): JEP 243: Java-Level JVM Compiler Interface

2015-09-17 Thread Magnus Ihse Bursie
On 2015-09-16 22:25, Christian Thalinger wrote: On Sep 16, 2015, at 10:11 AM, Magnus Ihse Bursie > wrote: On 2015-09-16 18:52, Christian Thalinger wrote: On Sep 16, 2015, at 2:57 AM, Magnus Ihse Bursie

Re: RFR (XXL): JEP 243: Java-Level JVM Compiler Interface

2015-09-17 Thread Christian Thalinger
Since there are no objections I’m going to push this… http://hg.openjdk.java.net/graal/graal-jvmci-9/hotspot/rev/6a6766a8cbbf > On Sep 16, 2015, at 3:05 PM, Christian Thalinger > wrote: > > I would like to add this change: > > diff -r 2134e08cc132

Re: RFR (XXL): JEP 243: Java-Level JVM Compiler Interface

2015-09-16 Thread Magnus Ihse Bursie
On 2015-09-14 09:24, Christian Thalinger wrote: The JEP itself can be found here: https://bugs.openjdk.java.net/browse/JDK-8062493 Here are the webrevs: http://cr.openjdk.java.net/~kvn/JVMCI/webrev.top/

Re: RFR (XXL): JEP 243: Java-Level JVM Compiler Interface

2015-09-16 Thread Christian Thalinger
> On Sep 16, 2015, at 2:57 AM, Magnus Ihse Bursie > wrote: > > On 2015-09-14 09:24, Christian Thalinger wrote: >> The JEP itself can be found here: >> >> https://bugs.openjdk.java.net/browse/JDK-8062493 >> >>

Re: RFR (XXL): JEP 243: Java-Level JVM Compiler Interface

2015-09-16 Thread Alan Bateman
On 16/09/2015 13:57, Magnus Ihse Bursie wrote: : * modules.xml: Make sure you get sign-off from a Jigsaw developer for modifying this file. The update to modules.xml looks fine. -Alan.

Re: RFR (XXL): JEP 243: Java-Level JVM Compiler Interface

2015-09-16 Thread Magnus Ihse Bursie
On 2015-09-16 18:52, Christian Thalinger wrote: On Sep 16, 2015, at 2:57 AM, Magnus Ihse Bursie wrote: On 2015-09-14 09:24, Christian Thalinger wrote: The JEP itself can be found here: https://bugs.openjdk.java.net/browse/JDK-8062493

Re: RFR (XXL): JEP 243: Java-Level JVM Compiler Interface

2015-09-16 Thread Christian Thalinger
> On Sep 16, 2015, at 10:11 AM, Magnus Ihse Bursie > wrote: > > On 2015-09-16 18:52, Christian Thalinger wrote: >>> On Sep 16, 2015, at 2:57 AM, Magnus Ihse Bursie >>> wrote: >>> >>> On 2015-09-14 09:24, Christian Thalinger

Re: RFR (XXL): JEP 243: Java-Level JVM Compiler Interface

2015-09-16 Thread Christian Thalinger
> On Sep 16, 2015, at 6:52 AM, Christian Thalinger > wrote: > > >> On Sep 16, 2015, at 2:57 AM, Magnus Ihse Bursie >> wrote: >> >> On 2015-09-14 09:24, Christian Thalinger wrote: >>> The JEP itself can be found here: >>> >>>

Re: RFR (XXL): JEP 243: Java-Level JVM Compiler Interface

2015-09-16 Thread Vladimir Kozlov
I updated top and hotspot webrev with latest (build) changes. Vladimir On 9/16/15 2:28 PM, Christian Thalinger wrote: On Sep 16, 2015, at 6:52 AM, Christian Thalinger wrote: On Sep 16, 2015, at 2:57 AM, Magnus Ihse Bursie

Re: RFR (XXL): JEP 243: Java-Level JVM Compiler Interface

2015-09-16 Thread Christian Thalinger
I would like to add this change: diff -r 2134e08cc132 src/share/vm/utilities/vmError.cpp --- a/src/share/vm/utilities/vmError.cppWed Sep 16 14:28:33 2015 -1000 +++ b/src/share/vm/utilities/vmError.cppWed Sep 16 15:04:02 2015 -1000 @@ -517,6 +517,10 @@ void