Request for review (XS): 8010116: Abstract_VM_Version::internal_vm_info_string() should recognize VS2010 and VS2012

[Krystal Mo]

Thank you, Zhengyu!

- Kris

On 2013/3/14 13:10, Zhengyu Gu wrote:
> Looks good to me.
>
> -Zhengyu
>
> On 3/14/2013 3:44 PM, Krystal Mo wrote:
>> Hi all,
>>
>> Could I get a review for this small fix, please?
>>
>> Webrev: http://cr.openjdk.java.net/~kmo/8010116/webrev.00/
>> CR: http://bugs.sun.com/view_bug.do?bug_id=8010116
>>
>> Description:
>>
>> Currently Abstract_VM_Version::internal_vm_info_string() recognizes 
>> MSVC versions from 5.0 to 9.0 (VS2008). Since JDK7 can be and is 
>> being built with VS2010, it should recognize that compiler (along 
>> with new VS2012), so that when its value appears in crash logs, it 
>> shouldn't say: "unknown MS VC++:1600", which is what can be seen in 
>> JDK7u's crash log.
>>
>> Thanks,
>> Kris
>