On Tue, 9 Apr 2024 17:00:56 GMT, Thomas Stuefe <stu...@openjdk.org> wrote:
>> 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
I had tried this, but got following error:
.../src/hotspot/os_cpu/aix_ppc/os_aix_ppc.cpp:438:40: error: no matching
function for call to 'p2i'
st->print("pc =" INTPTR_FORMAT " ", p2i(uc->uc_mcontext.jmp_context.iar));
^~~
.../src/hotspot/share/utilities/globalDefinitions.hpp:179:17: note: candidate
function not viable: no known conversion from 'const unsigned long long' to
'const volatile void *' for 1st argument; take the address of the argument with
&
inline intptr_t p2i(const volatile void* p) {
^
.../src/hotspot/share/oops/oopsHierarchy.hpp:169:17: note: candidate function
not viable: no known conversion from 'const unsigned long long' to 'narrowOop'
for 1st argument
inline intptr_t p2i(narrowOop o) {
^
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/18536#discussion_r1559128609
