RFR: JDK-8301378: CodeHeap has virtual methods that are not overridden
Tobias Hartmann
thartmann at openjdk.org
Tue Jan 31 08:10:07 UTC 2023
On Mon, 30 Jan 2023 18:24:52 GMT, Justin King <jcking at openjdk.org> wrote:
> Remove unused `virtual` specifiers on `CodeHeap` methods. Virtual dispatch is not free in C++ and usually incurs a vtable lookup unless the compiler can prove the type is final, which with the current setup is not feasible.
Unfortunately, this caused massive failures with the SA tests and we therefore had to back the change out (see [JDK-8301448](https://bugs.openjdk.org/browse/JDK-8301448)). @jcking please enable testing via Github actions, see https://openjdk.org/guide/#github-actions.
-------------
PR: https://git.openjdk.org/jdk/pull/12298
More information about the hotspot-compiler-dev
mailing list