Hi Thomas,

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.

Thanks,
David

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

Reply via email to