Need reviewer: CC_VER checks on compiler options

Kelly O'Hair kelly.ohair at oracle.com
Fri Apr 22 16:17:48 UTC 2011


On Apr 22, 2011, at 12:12 AM, David Holmes wrote:

> Hi Kelly,
> 
> Kelly O'Hair said the following on 04/22/11 07:33:
>> This started out as an annoyance around the use of -Wno-clobber on Linux when the gcc might
>> not support it. Turned into fixing several CC_VER checks in the makefiles:
>> 7038711: Fix CC_VER checks for compiler options, fix use of -Wno-clobber
>> http://cr.openjdk.java.net/~ohair/openjdk7/jdk7-build-ccver/webrev/
> 
> We can now drop the cross-compile check in jpeg/Makefile if you are going to make this conditional on the compiler version. The only reason we added this was because our ppc compiler is only 4.1.2
> 

I wondered about that, but wasn't sure. Wanted to make the lowest risk change at this point.

> So one thing I am confused about is the required gcc version. We had bumped it up to 4.3 because of this no-clobber usage. But now that use of that option is conditional does that mean we can lower the required gcc version again?

The 4.3.0 required was when we made Fedora 9 our official build system for jdk7.
I didn't think 4.3.0 had anything to do with no-clobber, but I could be wrong.

-kto

> 
> David
> 
> 




More information about the build-dev mailing list