RFR: 8258961: devirtualize SafePointNode::jvms()

Xin Liu xliu at openjdk.java.net
Wed Jan 6 09:32:57 UTC 2021


On Wed, 6 Jan 2021 09:15:24 GMT, Tobias Hartmann <thartmann 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.

yes, I did modify the JBS title. I found it's hard to hide out

-------------

PR: https://git.openjdk.java.net/jdk/pull/1899


More information about the hotspot-compiler-dev mailing list