review request for 6718830
Xiaobin Lu
Xiaobin.Lu at Sun.COM
Thu Jun 26 09:38:13 PDT 2008
Webrev: http://javaweb.sfbay/~xl116366/webrev/6718830/
6718830: Hotspot fails to build with gcc 4.3
Details:
Two of the incompatibility changes introduced in gcc 4.3 will affect the
success build of hotspot. The first change is the meaning of
"-Wconversion" flag. This flag will now emit warnings about every
implicit downcast such as from int to char or short, from double to
float, or int etc. While it would be nice to fix this everywhere in our
code base, I think it would be safe not to do so mainly because it will
affect a lot of type declaration of the fields in our classes. For
example, class symbolOopDesc, the "_length" field is declared as
"unsigned short", however, the methods of setting the length accepts
"int". Such cases widely spread across our code base. My fix is to
disable "-Wconversion" flag for build using GCC 4.3, the old meaning of
"-Wconversion" is regressed to "-Wtraditional-conversion" which is
available only for C and object C code.
The second change is the removal of Pre-ISO C++ headers such as
"iostream.h", instead the GCC 4.3 porting notes suggests to migrate to
include "iostream" and specify "using namespace std". This will affect
two places in our adlc code which uses "iostream" class in C++.
Verified by:
PRT
Successful build and test on OpenSuSE 11 (gcc 4.3.1)
Reviewed by:
Thanks,
-Xiaobin
More information about the hotspot-dev
mailing list