Time to rebase on top of JDK 9 repository?

[Timo Kinnunen]

I tried this but there's quite a lot of compile errors from constructs that were legal in C++11 but are not in C++14 that have been cleaned in the jdk9 already. For example many macros get interpreted as user-defined literals.
So the merge to get these is needed.


Sent from Outlook

    _____________________________
From: Timo Kinnunen <timo.kinnunen at gmail.com>
Sent: Tuesday, September 29, 2015 3:35 pm
Subject: RE: Time to rebase on top of JDK 9 repository?
To: Maurizio Cimadamore <maurizio.cimadamore at oracle.com>,  <valhalla-dev at openjdk.java.net>


              

I’m not sure what exactly would be sufficient, but biggest problems are with reorganized C/C++ files in jdk9/hotspot and jdk9/jdk repositories. I also have small changes to autoconf templates in the root jdk9 repository. They are easy enough to add if the build is otherwise functional.   

    

I could try with only the build changes included to see how far it goes.   

    

    

    

    

Sent from Mail for Windows 10   

    

        


From: Maurizio Cimadamore
Sent: Tuesday, September 29, 2015 15:10
To: timo.kinnunen at gmail.com;valhalla-dev at openjdk.java.net
Subject: Re: Time to rebase on top of JDK 9 repository?      

    

    

    

    

On 29/09/15 13:58, timo.kinnunen at gmail.com wrote:   

> Hi,   

>    

> I’ve been looking into compiling Valhalla with Visual Studio 2015 to try it out but I have hit a snag. See, I have made the needed changes and have a working build but I made the changes on top of the jdk9/jdk9 repository. I tried bringing the changes over to the valhalla/valhalla repository but the valhalla repository seems to be several months behind the JDK 9 repositories. There’s quite a lot of changes missing. Is it time to merge in the latest JDK 9 changes?   

>    

> I’m not an expert on mercurial nor familiar with the valhalla implementation details so I’m otherwise pretty much stuck.   

Yep - I agree it's time to merge - it's on my radar; as a short-term    

measure, is there any specific changeset you need pushed in order to get    

up and running with VS2015?   

    

Maurizio   

>    

>    

>    

> Sent from Mail for Windows 10