RFR: 8367325: [s390x] build failure due to JDK-8361376 [v2]
Martin Doerr
mdoerr at openjdk.org
Wed Sep 17 10:11:11 UTC 2025
On Wed, 17 Sep 2025 06:01:37 GMT, Amit Kumar <amitkumar at openjdk.org> wrote:
>> src/hotspot/cpu/s390/gc/shared/barrierSetNMethod_s390.cpp line 43:
>>
>>> 41: address inst_addr = get_barrier_start_address() + BarrierSetAssembler::PATCHABLE_INSTRUCTION_OFFSET;
>>> 42:
>>> 43: DEBUG_ONLY(Assembler::is_z_cfi(*((long*)inst_addr)));
>>
>> The cfi check needs an adjustment to point to the instruction start, right?
>
> Now It is pointing to `z_cfi`:
>
>
> (gdb) x/i inst_addr
> 0x3ffe500017a: cfi %r0,0
This line looks broken (already before this PR). Shouldn't it be something like `assert(Assembler::is_z_cfi...)`?
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/27213#discussion_r2354998084
More information about the hotspot-dev
mailing list