On Tue, 2 Apr 2024 16:14:12 GMT, Joachim Kern <jk...@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
>
> Joachim Kern has updated the pull request incrementally with one additional 
> commit since the last revision:
> 
>   version check not needed anymore

src/hotspot/os_cpu/aix_ppc/os_aix_ppc.cpp line 440:

> 438:   st->print("pc =" INTPTR_FORMAT "  ", (unsigned 
> long)uc->uc_mcontext.jmp_context.iar);
> 439:   st->print("lr =" INTPTR_FORMAT "  ", (unsigned 
> long)uc->uc_mcontext.jmp_context.lr);
> 440:   st->print("ctr=" INTPTR_FORMAT "  ", (unsigned 
> long)uc->uc_mcontext.jmp_context.ctr);

p2i

src/hotspot/os_cpu/aix_ppc/os_aix_ppc.cpp line 443:

> 441:   st->cr();
> 442:   for (int i = 0; i < 32; i++) {
> 443:     st->print("r%-2d=" INTPTR_FORMAT "  ", i, (unsigned 
> long)uc->uc_mcontext.jmp_context.gpr[i]);

p2i

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

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

Reply via email to