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 VS2015, since all
documentation I could find seemed to indicated that only 2015 and
updates 1-3 were released. If you have more info on this let me know!
My installation of 2015 was put in "Microsoft Visual Studio 14.0"
following the pattern of previous versions (12.0, 11.0, 10.0 etc), so I
think that would be the appropriate number here. Otherwise I think this
looks good.
/Erik
http://cr.openjdk.java.net/~lfoltan/bug_jdk8198312.1/webrev/
Thanks,
Lois
/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
http://cr.openjdk.java.net/~lfoltan/bug_jdk8198312/webrev/
bug link https://bugs.openjdk.java.net/browse/JDK-8198312
Testing: hs-tier(1-3), jdk-tier(1-3) complete
Thanks,
Lois