RFR: JDK-8288094: cleanup old _MSC_VER handling [v2]

Andrey Turbanov aturbanov at openjdk.java.net
Mon Jun 13 10:36:54 UTC 2022


On Mon, 13 Jun 2022 07:24:52 GMT, Matthias Baesken <mbaesken at openjdk.org> wrote:

>> We still handle at a number of places ancient historic _MSC_VER versions of Visual Studio releases e.g. pre VS2013 (VS2013 has _MSC_VER 1800).
>> This should be cleaned up, as long as it is not 3rd party code that we don't want to adjust.
>> 
>> Currently still supported ("valid") VS version are 2017+, see https://github.com/openjdk/jdk/blob/master/make/autoconf/toolchain_microsoft.m4 .
>> VALID_VS_VERSIONS="2019 2017 2022"
>> Even with adjusted toolchain m4 files, something older than VS2013 (also probably older than VS2015) would not be able to build jdk anymore.
>
> Matthias Baesken has updated the pull request incrementally with one additional commit since the last revision:
> 
>   use round directly

src/hotspot/share/adlc/main.cpp line 491:

> 489: }
> 490: 
> 491: // VS2005 has its own definition, identical to this one.

Let's adjust comment too. No need to mention VS2005.

-------------

PR: https://git.openjdk.org/jdk/pull/9105


More information about the core-libs-dev mailing list