Need reviewer - two fixes for one

Kelly O'Hair Kelly.Ohair at Sun.COM
Wed Jul 8 00:40:05 UTC 2009


Kelly O'Hair wrote:
> 
> Need reviewer.
> 
> 6855551: java -Xrunhprof crashes when running with classes compiled with 
> targed=7
>   Related to 6855180, assert logic could get segv, investigation found 
> next one.
> 
> 6858127: Missing -DNDEBUG on Linux and Windows native code compiles
>   The C/C++ macro NDEBUG when defined turns assert() macros into 
> nothingness.
>   These assert() macros are for debug builds, not product builds, when
>   an assert fails, it prints out information and typically exits the 
> process.

Just a clarification...  This -DNDEBUG option addition to compile lines
will likely result in a small reduction in code size, and a small performance
improvement on Linux and Windows, perhaps hard to measure.
I will verify that a JDK7 image built this way still serves as a basic
BOOT JDK successfully.
The product builds (optimized non-debug builds) should not rely on
native code standard assert() uses to do 'anything'.

-kto

> 
> Webrev:
>   http://cr.openjdk.java.net/~ohair/jdk7/jdk7-tl-6855551%2b6858127/webrev/
> 
> -kto
> 



More information about the build-dev mailing list