RFR 8056039: Hotspot does not compile with clang 3.4 on Linux

Mikael Gerdin mikael.gerdin at oracle.com
Thu Sep 11 06:56:04 UTC 2014


Thanks for the review Staffan!

/Mikael

On Wednesday 10 September 2014 20.43.29 Staffan Larsen wrote:
> Removed code is good code. Reviewed.
> 
> /Staffan
> 
> On 26 aug 2014, at 12:17, Mikael Gerdin <mikael.gerdin at oracle.com> wrote:
> > Hi all,
> > 
> > In order to get clang's (sometimes) more helpful error messages when
> > compiling I'd like to fix the few remaining places where clang fails to
> > compile Hotspot.
> > 
> > The culprit in this case was "local_vsnprintf" in os_linux.cpp, an unused
> > function which wasn't annotaded with the PRINTF_FORMAT macro.
> > Since the function was unused I decided to remove it instead, then I found
> > it in the other os_*nix.cpp files as well.
> > 
> > Digging into the Teamware history it looks like it first appeared in the
> > Solaris port because vsnprintf did not exist on some very old versions of
> > Solaris, so it was dynamically looked up through dlsym. For a few years
> > vsnprintf has been present in the Solaris header files, so I think it's
> > safe to remove the workaround now some 17 years later.
> > 
> > I also need a SCANF_FORMAT for an internal file, so I added that to
> > globalDefinitions.
> > 
> > Webrev: http://cr.openjdk.java.net/~mgerdin/8056039/webrev/
> > Buglink: https://bugs.openjdk.java.net/browse/JDK-8056039
> > 
> > Thanks
> > /Mikael



More information about the hotspot-dev mailing list