RFR(S): 8211124: HotSpot update for vm_version.cpp to recognise updated VS2017

[Kevin Walls]

Thanks both!


On 26/09/2018 13:36, Lois Foltan wrote:
> +1
> Lois
>
> On 9/26/2018 7:44 AM, David Holmes wrote:
>> Looks fine Kevin!
>>
>> Thanks,
>> David
>>
>> On 26/09/2018 6:46 AM, Kevin Walls wrote:
>>> Hi,
>>>
>>> I'd like to get a review for a simple update to make hotspot aware 
>>> of an additional Windows compiler version, for use in its long 
>>> version info string (and in hs_err crash reports).
>>>
>>> 8211124: HotSpot update for vm_version.cpp to recognise updated VS2017
>>> https://bugs.openjdk.java.net/browse/JDK-8211124
>>>
>>> Thanks
>>> Kevin
>>>
>>> bash-4.2$ hg diff
>>> diff -r a6bdb6d5f167 src/hotspot/share/runtime/vm_version.cpp
>>> --- a/src/hotspot/share/runtime/vm_version.cpp  Tue Sep 25 11:31:55 
>>> 2018 -0700
>>> +++ b/src/hotspot/share/runtime/vm_version.cpp  Wed Sep 26 03:34:17 
>>> 2018 -0700
>>> @@ -220,6 +220,8 @@
>>>           #define HOTSPOT_BUILD_COMPILER "MS VC++ 14.0 (VS2015)"
>>>         #elif _MSC_VER == 1912
>>>           #define HOTSPOT_BUILD_COMPILER "MS VC++ 15.5 (VS2017)"
>>> +      #elif _MSC_VER == 1913
>>> +        #define HOTSPOT_BUILD_COMPILER "MS VC++ 15.6 (VS2017)"
>>>         #else
>>>           #define HOTSPOT_BUILD_COMPILER "unknown MS VC++:" 
>>> XSTR(_MSC_VER)
>>>         #endif
>>> bash-4.2$
>>>
>>>
>