RFR(xs): 8171225: [aix] Build gtests on AIX 7.1 with xlC 12
david.holmes at oracle.com
Thu Dec 15 10:33:16 UTC 2016
On 15/12/2016 4:43 PM, 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.
These kind of problems have been reported in the past. The way we chose
to address them was to convert to use ASSERT_TRUE(i == NULL) rather than
apply casts to make the ASSERT_EQ(i, NULL) compile.
> 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
> (top level)
> 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