RFR: 8285487: AArch64: Do not generate unneeded trampolines for runtime calls [v6]
Evgeny Astigeevich
duke at openjdk.org
Thu Aug 11 21:19:07 UTC 2022
> Runtime calls are calls of non-compiled methods. Non-compiled methods stay forever in CodeCache. If they are always within the branch range, they don't need trampolines.
>
> This PR adds `is_always_within_branch_range(Address entry)`.
>
> Results from DaCapo: the total number of eliminated trampolines per a benchmark run
>
>
> +----------+--------+
> | avrora | 15491 |
> | batik | 75837 |
> | biojava | 13927 |
> | eclipse | 414143 |
> | fop | 119267 |
> | graphchi | 7665 |
> | jme | 8279 |
> | luindex | 56061 |
> | lusearch | 50277 |
> | pmd | 132719 |
> | sunflow | 10689 |
> | tomcat | 186967 |
> | xalan | 50349 |
> | zxing | 41497 |
> +----------+--------+
>
>
>
> Testing:
> - `tier1`...`tier2`: Passed
> - `compiler/c2/aarch64/TestTrampoline.java`: Passed
>
> Note: `compiler/c2/aarch64/TestTrampoline.java` requires the release build. This is because debug builds have the branch range set to 2M which causes always generation of trampolines.
Evgeny Astigeevich has updated the pull request incrementally with one additional commit since the last revision:
Clarify non-compiled method in comments
-------------
Changes:
- all: https://git.openjdk.org/jdk/pull/8403/files
- new: https://git.openjdk.org/jdk/pull/8403/files/4db9dfc9..55872c47
Webrevs:
- full: https://webrevs.openjdk.org/?repo=jdk&pr=8403&range=05
- incr: https://webrevs.openjdk.org/?repo=jdk&pr=8403&range=04-05
Stats: 1 line in 1 file changed: 0 ins; 0 del; 1 mod
Patch: https://git.openjdk.org/jdk/pull/8403.diff
Fetch: git fetch https://git.openjdk.org/jdk pull/8403/head:pull/8403
PR: https://git.openjdk.org/jdk/pull/8403
More information about the hotspot-compiler-dev
mailing list