RFR: 8224214: [AIX] Remove support for legacy xlc compiler
Baesken, Matthias
matthias.baesken at sap.com
Thu Aug 29 15:41:45 UTC 2019
Hi Martin, I agree about the m4 files .
New webrev , this additionally touches toolchain.m4 and flags-cflags.m4
http://cr.openjdk.java.net/~mbaesken/webrevs/8224214.1/
Thanks, Matthias
> -----Original Message-----
> From: Doerr, Martin <martin.doerr at sap.com>
> Sent: Donnerstag, 29. August 2019 16:19
> To: Baesken, Matthias <matthias.baesken at sap.com>; 'hotspot-
> dev at openjdk.java.net' <hotspot-dev at openjdk.java.net>; 'ppc-aix-port-
> dev at openjdk.java.net' <ppc-aix-port-dev at openjdk.java.net>
> Subject: RE: RFR: 8224214: [AIX] Remove support for legacy xlc compiler
>
> Hi Matthias,
>
> nice cleanup. Looks good to me.
>
> We can also require availability of xlclang++ in toolchain.m4. I think some of
> the changes only work with this compiler.
>
> Thanks,
> Martin
>
>
> > -----Original Message-----
> > From: hotspot-dev <hotspot-dev-bounces at openjdk.java.net> On Behalf
> Of
> > Baesken, Matthias
> > Sent: Donnerstag, 29. August 2019 15:41
> > To: 'hotspot-dev at openjdk.java.net' <hotspot-dev at openjdk.java.net>;
> > 'ppc-aix-port-dev at openjdk.java.net' <ppc-aix-port-
> dev at openjdk.java.net>
> > Subject: RFR: 8224214: [AIX] Remove support for legacy xlc compiler
> >
> > Hello, please review the following change .
> > For OpenJDK 13 we've moved to XLC 16 as required compiler.
> > However we have still a lot of workarounds and checks in the codebase
> for
> > the older xlc compilers.
> > This changes removes such changes .
> >
> > Additionally it adjusts the compiler version check in
> > hotspot/share/utilities/globalDefinitions_xlc.hpp
> > and 2 typos in os_aix are fixed .
> >
> >
> > When 8224214 was created a while ago , it was discussed on the mailing
> list
> > :
> >
> > "we still set some '-qlanglvl' options for C++ which aren't supported by the
> > new compiler [xlc16/xlclang++] either" .
> > Those options generated lots of warnings , so they were removed
> already
> > so no need to remove them in this change .
> >
> > ( In jdk11 which is built with xlc12 they can still be found :
> > flags-cflags.m4:540: -qalias=noansi -qstrict -qtls=default -qlanglvl=c99vla \
> > flags-cflags.m4:541: -qlanglvl=noredefmac -qnortti -qnoeh -qignerrno"
> > )
> >
> >
> > Bug/webrev :
> >
> > https://bugs.openjdk.java.net/browse/JDK-8224214
> >
> > http://cr.openjdk.java.net/~mbaesken/webrevs/8224214.0/
> >
> >
> > Thanks, Matthias
More information about the ppc-aix-port-dev
mailing list