<div dir="ltr">Hi Eric,<div class="gmail_extra"><br><div class="gmail_quote">On Wed, Nov 4, 2015 at 11:46 AM, 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:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">(adding build-dev as well)<br>
<span class=""><br>
On 2015-11-03, Volker Simonis wrote:<br>
> Hi Erik,<br>
><br>
> while I agree that this is a nice clean up I want to second Thomas in<br>
> his attempt to save the VS 2010 build. I understand that the official<br>
> Oracle build tool chain for jdk9 on Windows is VS 2013, but this may<br>
> be different for others. I also think that we shouldn't sacrifice the<br>
> possibility to use  VS 2010 without any hard requirements like for<br>
> example the need to use new language features which are not supported<br>
> in the old version.<br>
<br>
</span>Hi Volker,<br>
<br>
I understand that you guys want to keep building VS2010 but I know that<br>
you also understand that we want to get rid of this cruft from the GC<br>
code.<br>
<span class=""><br>
> If you and others still insist in doing this change, I'd strongly vote<br>
> to first completely disable the support for VS 2010 in the 'configure'<br>
> process before we start to do source code changes.<br>
><br>
> Instead, I'd prefer if you could just change the code from:<br>
><br>
> #ifdef _MSC_VER<br>
><br>
> to:<br>
><br>
> #if _MSC_VER < 1800<br>
><br>
> like it was already done for<br>
> <a href="https://bugs.openjdk.java.net/browse/JDK-8137329" rel="noreferrer" target="_blank">https://bugs.openjdk.java.net/browse/JDK-8137329</a>. This makes the<br>
> dependency on a specific compiler version explicit (although it would<br>
> hardly count as a "clean up" any more :)<br>
<br>
</span>Maybe we can come up with something that removes these #pragmas from the<br>
GC code while still allowing you to compile with VS2010. What do you<br>
think about the following suggestions:<br>
- We change the Makefiles to add the flag /Wd4355 if we use VS2010 or<br>
  older. This will results in the warning being disabled for all files<br>
  (one could potentially also disable the warning per file).<br>
- We add the code:<br>
  #if _MSC_VER < 1000<br>
    #pragma warning( disable:4355 )<br>
  #endif // _MSC_VER < 1800<br>
  to globalDefinitions_visCPP.hpp. This file gets included in<br>
  globalDefinitions.hpp which in turn gets included basically<br>
  everywhere. This will turn off the warning for essentially all files.<br>
<br>
I'm leaning more towards the Makefile solution since that is where it<br>
makes most sense to me to disable warnings. OTOH,<br>
globalDefinitions_visCPP.hpp already disables quite a few warnings...<br>
<br></blockquote><div><br></div><div>Thanks!</div><div><br></div><div>Both solutions are fine, I would prefer adding them to globalDefinitions_visCPP.hpp,</div><div>which already contains a number of other workarounds for older MSVC versions.</div><div><br></div><div>Kind Regards, Thomas</div><div><br></div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">
build-dev, I don't know if you guys have any other suggestion?<br>
<br>
Thanks,<br>
Erik<br>
<div class=""><div class="h5"><br>
> Regards,<br>
> Volker<br>
><br>
><br>
><br>
><br>
> On Tue, Nov 3, 2015 at 3:32 PM, Thomas Stüfe <<a href="mailto:thomas.stuefe@gmail.com">thomas.stuefe@gmail.com</a>> wrote:<br>
> > Hi,<br>
> ><br>
> > one question, does this mean this would not build anymore with Visual Studio<br>
> > 2010?<br>
> ><br>
> > According to :<br>
> > <a href="https://wiki.openjdk.java.net/display/Build/Supported+Build+Platforms" rel="noreferrer" target="_blank">https://wiki.openjdk.java.net/display/Build/Supported+Build+Platforms</a> ,<br>
> > VS2010 is still supported build platform.<br>
> ><br>
> > Also there seems to be enough interest in the community about keeping the<br>
> > VS2010 build alive, see<br>
> > <a href="http://mail.openjdk.java.net/pipermail/hotspot-runtime-dev/2015-October/015978.html" rel="noreferrer" target="_blank">http://mail.openjdk.java.net/pipermail/hotspot-runtime-dev/2015-October/015978.html</a><br>
> ><br>
> > We (SAP) unfortunately still use Visual Studio 2010 for our VM, it would be<br>
> > nice if that would continue to work.<br>
> ><br>
> > Best Regards, Thomas<br>
> ><br>
> ><br>
> ><br>
> > On Mon, Nov 2, 2015 at 5:17 PM, Erik Helin <<a href="mailto:erik.helin@oracle.com">erik.helin@oracle.com</a>> wrote:<br>
> >><br>
> >> On 2015-11-02, Thomas Schatzl wrote:<br>
> >> > Hi all,<br>
> >> ><br>
> >> >   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>
> >> Looks good, Reviewed.<br>
> >><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<br>
> >> > > 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>
> ><br>
> ><br>
</div></div></blockquote></div><br></div></div>