RFR: JDK-8210920 Native C++ tests are not using CXXFLAGS

Erik Joelsson erik.joelsson at oracle.com
Wed Sep 19 22:48:27 UTC 2018


Looks ok.

/Erik


On 2018-09-19 13:55, Magnus Ihse Bursie wrote:
> Native C++ tests are not using CXXFLAGS. This is mostly apparent on 
> solaris, where CC and cc are quite different. Fixing this made some 
> new warnings appear on C++ tests for solstudio/gcc.
>
> Also, when fixing this, it became apparent that the 
> *FLAGS_TESTLIB/TESTEXE is just clumsy. In the long term, these super 
> combined flags should be broken up. In the short term, just use 
> *FLAGS_JDK explicitly, since that's what we're doing anyway.
>
> Bug: https://bugs.openjdk.java.net/browse/JDK-8210920
> WebRev: 
> http://cr.openjdk.java.net/~ihse/JDK-8210920-fix-testlibs-CXX-flags/webrev.01
>
> /Magnus
>




More information about the build-dev mailing list