[riscv-port-jdk11u:riscv-port] RFR: 8294366: RISC-V: Partially mark out incompressible regions [v2]
Dingli Zhang
dzhang at openjdk.org
Thu Oct 17 02:45:28 UTC 2024
On Thu, 17 Oct 2024 02:11:06 GMT, Fei Yang <fyang at openjdk.org> wrote:
>> Dingli Zhang has updated the pull request incrementally with one additional commit since the last revision:
>>
>> fix build_frame
>
> src/hotspot/cpu/riscv/c1_MacroAssembler_riscv.cpp line 324:
>
>> 322: // must ensure that this first instruction is a J, JAL or NOP.
>> 323: // Make it a NOP.
>> 324: IncompressibleRegion ir(this); // keep the nop as 4 bytes for patching.
>
> I think you should move the `assert_alignment(pc())` in `C1_MacroAssembler::verified_entry` here before this nop. Seens that assertion was miss-placed by previous backport PR: https://github.com/openjdk/riscv-port-jdk11u/pull/40
Thanks for the review, fixed.
-------------
PR Review Comment: https://git.openjdk.org/riscv-port-jdk11u/pull/43#discussion_r1804028804
More information about the riscv-port-dev
mailing list