Re: (11) RFR (S) JDK-8198312: VS2017: Upgrade HOTSPOT_BUILD_COMPILER in vm_version.cpp

2018-02-23 Thread Erik Joelsson
Looks good! /Erik On 2018-02-23 11:39, Lois Foltan wrote: On 2/23/2018 2:31 PM, Erik Joelsson wrote: On 2018-02-23 11:16, Lois Foltan wrote: On 2/23/2018 1:05 PM, Erik Joelsson wrote: Hello Lois, This looks good, but I would suggest to also add 1900 for VS2015, for completeness. Thanks

Re: (11) RFR (S) JDK-8198312: VS2017: Upgrade HOTSPOT_BUILD_COMPILER in vm_version.cpp

2018-02-23 Thread Lois Foltan
On 2/23/2018 2:31 PM, Erik Joelsson wrote: On 2018-02-23 11:16, Lois Foltan wrote: On 2/23/2018 1:05 PM, Erik Joelsson wrote: Hello Lois, This looks good, but I would suggest to also add 1900 for VS2015, for completeness. Thanks for the review Erik!  I have updated the webrev to add 1900,

Re: (11) RFR (S) JDK-8198312: VS2017: Upgrade HOTSPOT_BUILD_COMPILER in vm_version.cpp

2018-02-23 Thread Lois Foltan
Thanks again for the review Erik! Lois On 2/23/2018 2:44 PM, Erik Joelsson wrote: Looks good! /Erik On 2018-02-23 11:39, Lois Foltan wrote: On 2/23/2018 2:31 PM, Erik Joelsson wrote: On 2018-02-23 11:16, Lois Foltan wrote: On 2/23/2018 1:05 PM, Erik Joelsson wrote: Hello Lois, This

Re: (11) RFR (S) JDK-8198312: VS2017: Upgrade HOTSPOT_BUILD_COMPILER in vm_version.cpp

2018-02-23 Thread Erik Joelsson
On 2018-02-23 11:16, Lois Foltan wrote: On 2/23/2018 1:05 PM, Erik Joelsson wrote: Hello Lois, This looks good, but I would suggest to also add 1900 for VS2015, for completeness. Thanks for the review Erik!  I have updated the webrev to add 1900, however, I couldn't find a release # for

Re: (11) RFR (S) JDK-8198312: VS2017: Upgrade HOTSPOT_BUILD_COMPILER in vm_version.cpp

2018-02-23 Thread Lois Foltan
On 2/23/2018 1:05 PM, Erik Joelsson wrote: Hello Lois, This looks good, but I would suggest to also add 1900 for VS2015, for completeness. Thanks for the review Erik!  I have updated the webrev to add 1900, however, I couldn't find a release # for VS2015, since all documentation I could

Re: (11) RFR (S) JDK-8198312: VS2017: Upgrade HOTSPOT_BUILD_COMPILER in vm_version.cpp

2018-02-23 Thread Erik Joelsson
Hello Lois, This looks good, but I would suggest to also add 1900 for VS2015, for completeness. /Erik On 2018-02-23 09:54, Lois Foltan wrote: Please review this small fix to set HOTSPOT_BUILD_COMPILER correctly for VS2017. open webrev at