RFR: 8367692: RISC-V: Align post call nop [v7]
Robbin Ehn
rehn at openjdk.org
Tue Sep 30 15:14:39 UTC 2025
> Hi please, consider.
>
> As ziccif require instructions to natural aligned to be atomic the 4 byte post call nop must be aligned.
> But I don't want to add a c.nop(2b) to align the nop(4b) which means the jal(r) must also be algined.
> As we have no utility to aligned the end of an instruction sequence the call it self is aligned and uses only 4 byte instructions. Only in the case where we could use an two c-instruction we may loose space.
>
> Thanks, Robbin
Robbin Ehn has updated the pull request with a new target base due to a merge or a rebase. The incremental webrev excludes the unrelated changes brought in by the merge/rebase. The pull request contains eight additional commits since the last revision:
- Merge branch 'master' into align_post_nops
- Merge branch 'master' into align_post_nops
- Use relocation spec as marker.
- Revert "Removed relocate for post call nops"
This reverts commit 033ecd4ab3d62c1ccc7845d8bc9e62fc6574e05d.
- Removed relocate for post call nops
- Merge branch 'master' into align_post_nops
- Review comments
- init
-------------
Changes:
- all: https://git.openjdk.org/jdk/pull/27467/files
- new: https://git.openjdk.org/jdk/pull/27467/files/76c4f861..aedba042
Webrevs:
- full: https://webrevs.openjdk.org/?repo=jdk&pr=27467&range=06
- incr: https://webrevs.openjdk.org/?repo=jdk&pr=27467&range=05-06
Stats: 1972 lines in 99 files changed: 908 ins; 305 del; 759 mod
Patch: https://git.openjdk.org/jdk/pull/27467.diff
Fetch: git fetch https://git.openjdk.org/jdk.git pull/27467/head:pull/27467
PR: https://git.openjdk.org/jdk/pull/27467
More information about the hotspot-dev
mailing list