RFR: JDK-8329257: AIX: Switch HOTSPOT_TOOLCHAIN_TYPE from xlc to gcc

Joachim Kern jkern at openjdk.org
Tue Apr 2 11:26:00 UTC 2024


On Fri, 29 Mar 2024 07:39:06 GMT, Thomas Stuefe <stuefe at openjdk.org> wrote:

>> As of [JDK-8325880](https://bugs.openjdk.org/browse/JDK-8325880), building the JDK requires version 17 of IBM Open XL C/C++ (xlc). This is in effect clang by another name, and it uses the clang toolchain in the JDK build. Thus the old xlc toolchain was removed by [JDK-8327701](https://bugs.openjdk.org/browse/JDK-8327701).
>> Now we also switch the HOTSPOT_TOOLCHAIN_TYPE from xlc to gcc, removing the last xlc rudiment.
>> This means merging the AIX specific content of utilities/globalDefinitions_xlc.hpp and utilities/compilerWarnings_xlc.hpp into the corresponding gcc files on the on side and removing the defined(TARGET_COMPILER_xlc) blocks in the code, because the defined(TARGET_COMPILER_gcc) blocks work out of the box for the new AIX compiler.
>> The rest of the changes are needed because of using utilities/compilerWarnings_gcc.hpp the compiler is much more nagging about ill formatted printf
>
> src/hotspot/share/utilities/globalDefinitions_gcc.hpp line 62:
> 
>> 60: #include <errno.h>
>> 61: 
>> 62: #if defined(LINUX) || defined(_ALLBSD_SOURCE) || defined(_AIX)
> 
> What else is left? Could we just remove this line altogether now?

I cannot answer this question.
If this line is now obsolete it was also obsolete before including AIX, because AIX didn't use this file beforehand.

-------------

PR Review Comment: https://git.openjdk.org/jdk/pull/18536#discussion_r1547667349


More information about the build-dev mailing list