RFR: 8284578: Relax InterpreterCodelet stub alignment
Aleksey Shipilev
shade at openjdk.java.net
Wed Apr 13 09:15:15 UTC 2022
On Mon, 11 Apr 2022 22:56:55 GMT, Dean Long <dlong at openjdk.org> wrote:
> Does your change make this recent change obsolete, or is it still needed?
>
> https://github.com/openjdk/jdk/blob/026b85303c01326bc49a1105a89853d7641fcd50/src/hotspot/share/interpreter/templateInterpreter.cpp#L53
Checked, the additional slack space for alignment is still needed for the `compiler/arguments/TestCodeEntryAlignment.java` that tries large `CodeEntryAlignment`-s. `CodeEntryAlignment*2` is too much. `HeapWordSize * 2` is not enough. `HeapWordSize + CodeEntryAlignment` fits the intent of the patch. See new commit.
-------------
PR: https://git.openjdk.java.net/jdk/pull/8159
More information about the hotspot-dev
mailing list