<div dir="ltr">Hi,<div><br></div><div>one question, does this mean this would not build anymore with Visual Studio 2010? </div><div><br></div><div>According to : <a href="https://wiki.openjdk.java.net/display/Build/Supported+Build+Platforms">https://wiki.openjdk.java.net/display/Build/Supported+Build+Platforms</a> , VS2010 is still supported build platform.</div><div><br></div><div>Also there seems to be enough interest in the community about keeping the VS2010 build alive, see <a href="http://mail.openjdk.java.net/pipermail/hotspot-runtime-dev/2015-October/015978.html">http://mail.openjdk.java.net/pipermail/hotspot-runtime-dev/2015-October/015978.html</a></div><div><br></div><div>We (SAP) unfortunately still use Visual Studio 2010 for our VM, it would be nice if that would continue to work.<br></div><div><br></div><div>Best Regards, Thomas</div><div><br></div><div><br></div></div><div class="gmail_extra"><br><div class="gmail_quote">On Mon, Nov 2, 2015 at 5:17 PM, Erik Helin <span dir="ltr"><<a href="mailto:erik.helin@oracle.com" target="_blank">erik.helin@oracle.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class="">On 2015-11-02, Thomas Schatzl wrote:<br>
> Hi all,<br>
><br>
</span><span class="">>   Erik H. made me aware of another pragma 4355 in nmethod.cpp that can<br>
> be removed.<br>
><br>
> I updated the webrev in place because of the kind of change.<br>
<br>
</span>Looks good, Reviewed.<br>
<div class="HOEnZb"><div class="h5"><br>
> Thanks,<br>
>   Thomas<br>
><br>
> On Mon, 2015-11-02 at 16:14 +0100, Thomas Schatzl wrote:<br>
> > Hi all,<br>
> ><br>
> >   can I have reviews for the following change that removes some pragmas<br>
> > in GC code?<br>
> > With the move to newer MSVC, they are not needed any more.<br>
> ><br>
> > CR:<br>
> > <a href="https://bugs.openjdk.java.net/browse/JDK-8141134" rel="noreferrer" target="_blank">https://bugs.openjdk.java.net/browse/JDK-8141134</a><br>
> > Webrev:<br>
> > <a href="http://cr.openjdk.java.net/~tschatzl/8141134/webrev/" rel="noreferrer" target="_blank">http://cr.openjdk.java.net/~tschatzl/8141134/webrev/</a><br>
> > Testing:<br>
> > jprt<br>
> ><br>
> > Thanks,<br>
> >   Thomas<br>
> ><br>
> ><br>
><br>
><br>
</div></div></blockquote></div><br></div>