RFR(xs): 8171225: [aix] Build gtests on AIX 7.1 with xlC 12

Erik Joelsson erik.joelsson at oracle.com
Thu Dec 15 09:00:14 UTC 2016

Hello Thomas,

Build changes look ok. Please note that the configure changes requires 
synchronized changes in Oracle closed configure so will need an Oracle 
sponsor to push. I assume this is intended for jdk9/hs. I will be happy 
to push it for you once the change has been cleared to go in.


On 2016-12-15 07:43, Thomas Stüfe wrote:
> Hi all,
> please review this small change. It fixes the gtest build on AIX and
> enables it by default.
> Note that even though this is a fix for AIX, a cast needed to be added to
> shared test coding. This is because xlC struggles with certain template
> expansions and I had to help it by providing an explicit cast.
> Because linker options were changed as well, this unfortunately this
> spreads over two forest parts, so two webrevs were needed.
> Issue: https://bugs.openjdk.java.net/browse/JDK-8171225
> Webrevs:
> (hotspot)
> http://cr.openjdk.java.net/~stuefe/webrevs/8171225-aix-build-gtests/webrev.00/webrev/
> (top level)
> http://cr.openjdk.java.net/~stuefe/webrevs/8171225-aix-build-gtests/toplevel-webrev.00/webrev/
> Note that the toplevel change contains the newly generated configure.sh. I
> was not sure if that was needed, but it is included for convenience.
> Kind Regards, Thomas

More information about the hotspot-dev mailing list