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

[Erik Joelsson]

Yes, naturally.

/Erik


On 2016-12-15 11:06, Thomas Stüfe wrote:
> Hi Erik,
>
> thank you! I would have needed a sponsor for the hotspot change in any 
> case, would sponsor this too?
>
> Thomas
>
> On Thu, Dec 15, 2016 at 10:00 AM, Erik Joelsson 
> <erik.joelsson at oracle.com <mailto:erik.joelsson at oracle.com>> wrote:
>
>     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.
>
>     /Erik
>
>
>     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
>         <https://bugs.openjdk.java.net/browse/JDK-8171225>
>         Webrevs:
>         (hotspot)
>         http://cr.openjdk.java.net/~stuefe/webrevs/8171225-aix-build-gtests/webrev.00/webrev/
>         <http://cr.openjdk.java.net/%7Estuefe/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/
>         <http://cr.openjdk.java.net/%7Estuefe/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
>
>
>