graal and jdk8

[Lukas Stadler]

Can you try "mx clean"? The LogicNode class was moved recently, maybe a .class file is wasn't deleted.

- Lukas

On Feb 13, 2013, at 22:24 , "Venkatachalam, Vasanth" <Vasanth.Venkatachalam at amd.com> wrote:

> Okay, I updated and got the fixes. I’m seeing a new set of errors when I build with JDK-8.
> Here are a few snippets. I’ve attached the entire error log for reference.
> 
> /home/tester/vasanth/graal/graal/graal/com.oracle.graal.nodes/src/com/oracle/graal/nodes/java/InstanceOfNode.java:35: error: reference to LogicNode is ambiguous
> public final class InstanceOfNode extends LogicNode implements Canonicalizable, Lowerable, Virtualizable {
>                                          ^
>  both class com.oracle.graal.nodes.calc.LogicNode in com.oracle.graal.nodes.calc and class com.oracle.graal.nodes.LogicNode in com.oracle.graal.nodes match
> 
> 
> /home/tester/vasanth/graal/graal/graal/com.oracle.graal.nodes/src/com/oracle/graal/nodes/java/InstanceOfNode.java:104: error: incompatible types: LogicConstantNode cannot be converted to LogicNode
>            return LogicConstantNode.contradiction(graph());
> 
> /home/tester/vasanth/graal/graal/graal/com.oracle.graal.nodes/src/com/oracle/graal/nodes/java/InstanceOfDynamicNode.java:48: error: constructor LogicNode in class LogicNode cannot be applied to given types;
>    public InstanceOfDynamicNode(ValueNode mirror, ValueNode object) {
>                                                                     ^
>  required: Kind,ValueNode,ValueNode
> 
> 
> 
> From: Gilles Duboscq [mailto:gilwooden at gmail.com]
> Sent: Wednesday, February 13, 2013 3:05 PM
> To: Venkatachalam, Vasanth
> Cc: Thomas Wuerthinger; graal-dev at openjdk.java.net
> Subject: Re: graal and jdk8
> 
> Yes, they should be available i can see them in http://hg.openjdk.java.net/graal/graal
> 
> What revision are you currently at? (hg summary)
> Are you sure you pulled from the OpenJDK repository? (hg pull http://hg.openjdk.java.net/graal/graal)
> 
> On Wed, Feb 13, 2013 at 10:01 PM, Venkatachalam, Vasanth <Vasanth.Venkatachalam at amd.com<mailto:Vasanth.Venkatachalam at amd.com>> wrote:
> Hi, have the changes for that fix been checked in?
> 
> When I do an "hg update" under the root graal directory, I'm getting a "0 files modified" message.
> 
> Vasanth
> 
> -----Original Message-----
> From: graal-dev-bounces at openjdk.java.net<mailto:graal-dev-bounces at openjdk.java.net> [mailto:graal-dev-bounces at openjdk.java.net<mailto:graal-dev-bounces at openjdk.java.net>] On Behalf Of Gilles Duboscq
> Sent: Wednesday, February 13, 2013 1:31 PM
> To: Thomas Wuerthinger
> Cc: graal-dev at openjdk.java.net<mailto:graal-dev at openjdk.java.net>
> Subject: Re: graal and jdk8
> 
> The changes are currently propagating to http://hg.openjdk.java.net/graal/graal
> Note that to run with a JDK8 you will probably need to add -XX:+UnlockDiagnosticVMOptions -XX:+EnableInvokeDynamic For example "mx vm -XX:+UnlockDiagnosticVMOptions -XX:+EnableInvokeDynamic -version". This applies to both graal and server vm.
> 
> - Gilles
> 
> 
> On Wed, Feb 13, 2013 at 8:24 PM, Thomas Wuerthinger < thomas.wuerthinger at oracle.com<mailto:thomas.wuerthinger at oracle.com>> wrote:
> 
>> Yes, thanks for pointing that out - we will push that temporary fix asap!
>> - thomas
>> 
>> 
>> On Feb 13, 2013, at 11:05 AM, Remi Forax <forax at univ-mlv.fr<mailto:forax at univ-mlv.fr>> wrote:
>> 
>>> On 02/13/2013 08:04 PM, Venkatachalam, Vasanth wrote:
>>>> Are there any workarounds for this while the issue is being addressed?
>>> 
>>> a quick and dirty patch is to add a method isDefault that always
>>> return
>> false in  CodeExecutableElement.java
>>> 
>>>> 
>>>> Thanks,
>>>> 
>>>> Vasanth
>>> 
>>> cheers,
>>> Rémi
>>> 
>>>> 
>>>> -----Original Message-----
>>>> From: Thomas Wuerthinger [mailto:thomas.wuerthinger at oracle.com<mailto:thomas.wuerthinger at oracle.com>]
>>>> Sent: Wednesday, February 13, 2013 11:35 AM
>>>> To: Venkatachalam, Vasanth
>>>> Cc: graal-dev at openjdk.java.net<mailto:graal-dev at openjdk.java.net>
>>>> Subject: Re: graal and jdk8
>>>> 
>>>> OK thanks, we will investigate this. It has to do with the
>> annotation-based source code generation we use for parts of Truffle.
>> We are currently thinking of splitting this part of Graal into a
>> separate repository, because for many purposes of Graal (including the
>> ones of Project Sumatra) Truffle is not necessary.
>>>> 
>>>> - thomas
>>>> 
>>>> On Feb 13, 2013, at 9:27 AM, "Venkatachalam, Vasanth" <
>> Vasanth.Venkatachalam at amd.com<mailto:Vasanth.Venkatachalam at amd.com>> wrote:
>>>> 
>>>>> Hi Thomas,
>>>>> 
>>>>> When I try  to build with jdk-8, I get the following error:
>>>>> 
>>>>> Compiling Java sources for com.oracle.truffle.codegen.processor
>>>>> with
>> javac...
>>>>> warning: [options] bootstrap class path not set in conjunction
>>>>> with -source 1.7
>>>>> /home/tester/vasanth/graal/graal/graal/com.oracle.truffle.codegen.
>>>>> proc
>>>>> 
>> essor/src/com/oracle/truffle/codegen/processor/ast/CodeExecutableElement.java:34:
>> error: CodeExecutableElement is not abstract and does not override
>> abstract method isDefault() in ExecutableElement public class
>> CodeExecutableElement extends CodeElement<Element> implements
>> WritableExecutableElement {
>>>>>      ^
>>>>> 1 error
>>>>> 
>>>>> Here's the java version I am using:
>>>>> 
>>>>> tester at see-anna1:~/Downloads$ jdk1.8.0/bin/java -version java
>>>>> version "1.8.0-ea"
>>>>> Java(TM) SE Runtime Environment (build 1.8.0-ea-b74) Java
>>>>> HotSpot(TM) 64-Bit Server VM (build 25.0-b15, mixed mode)
>>>>> tester at see-anna1:~/Downloads$
>>>>> 
>>>>> -----Original Message-----
>>>>> From: Thomas Wuerthinger [mailto:thomas.wuerthinger at oracle.com<mailto:thomas.wuerthinger at oracle.com>]
>>>>> Sent: Wednesday, February 13, 2013 11:13 AM
>>>>> To: Venkatachalam, Vasanth
>>>>> Cc: graal-dev at openjdk.java.net<mailto:graal-dev at openjdk.java.net>
>>>>> Subject: Re: graal and jdk8
>>>>> 
>>>>> Vasanth,
>>>>> 
>>>>> We are not aware of any architectural problems that might prevent
>> Graal from running on a JDK8 host VM. Could you please give more
>> details on your setup and error message? We will resolve any
>> incompatibilities with
>> JDK8 asap.
>>>>> 
>>>>> Thanks, thomas
>>>>> 
>>>>> On Feb 13, 2013, at 9:03 AM, "Venkatachalam, Vasanth" <
>> Vasanth.Venkatachalam at amd.com<mailto:Vasanth.Venkatachalam at amd.com>> wrote:
>>>>> 
>>>>>> Hi,
>>>>>> 
>>>>>> I noticed that I had to use a JDK7 VM to build Graal.
>>>>>> I wasn't able to build it with JDK8. Is there a workaround for
>>>>>> this,
>> or does Graal require a JDK7 host VM?
>>>>>> 
>>>>>> If Graal only supports JDK7, are there plans to make it
>>>>>> compatible
>> with JDK8?
>>>>>> We're looking to leverage Graal for the Sumatra project
>>>>>> (http://openjdk.java.net/projects/sumatra/)
>>>>>> This work requires being able to compile JDK8 lambda constructs
>>>>>> for
>> the GPU.
>>>>>> 
>>>>>> Vasanth
>>>>> 
>>>>> 
>>>> 
>>>> 
>>> 
>> 
>> 
>