On Wed, 10 May 2023 13:59:15 GMT, JoKern65 <d...@openjdk.org> wrote:

>> The new xlc17 compiler should be supported to build OpenJDK on AIX. This 
>> compiler, compared to the currently supported xlc16, has a significantly 
>> more recent clang (xlc 17.1.1 uses clang 15) included.
>> 1. Because the frontend interface of the new compiler (c-flags, Ld-Flags) 
>> has changed from an xlc to a clang interface we decided to use the clang 
>> toolchain for the new xlc17 compiler.
>> 2. Unfortunately, the system headers are mainly unchanged, so they do not 
>> harmonize with the src/hotspot/share/utilities/globalDefinitions_gcc.hpp 
>> which would be used if we totally switch to clang toolchain. So we keep the 
>> HOTSPOT_TOOLCHAIN_TYPE=xlc
>> 3. In src/hotspot/share/utilities/globalDefinitions_xlc.hpp we introduce a 
>> new define AIX_XLC_GE_17 which is set if we build with the new xlc17 on AIX. 
>> This define will be used in following PRs.
>
> JoKern65 has updated the pull request incrementally with two additional 
> commits since the last revision:
> 
>  - revert accidantially changed mode bits
>  - I followed the proposals

Marked as reviewed by mbaesken (Reviewer).

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

PR Review: https://git.openjdk.org/jdk/pull/13898#pullrequestreview-1420735558

Reply via email to