RFR: 8258961: devirtualize SafePointNode::jvms() and hide out set_jvms()
Tobias Hartmann
thartmann at openjdk.java.net
Wed Jan 6 09:17:57 UTC 2021
On Mon, 28 Dec 2020 09:48:29 GMT, Xin Liu <xliu at openjdk.org> wrote:
> SafePointNode::jvms() declares virtual but has never been overridden. It seems unnecessary.
> The member variable _jvms declares 'JVMState* const', but set_jvms() updates its value anyway.
> Geting rid of the const qualifier and hides if out using protected.
@navyxliu There is a title mismatch between the PR and the JBS issue that you need to fix before integration.
-------------
PR: https://git.openjdk.java.net/jdk/pull/1899
More information about the hotspot-compiler-dev
mailing list