RFR: 8265453: SafepointMechanism::should_process() should receive JavaThread*
Daniel D.Daugherty
dcubed at openjdk.java.net
Wed Apr 21 19:17:39 UTC 2021
On Wed, 21 Apr 2021 14:47:13 GMT, Patricio Chilano Mateo <pchilanomate at openjdk.org> wrote:
> Hi,
>
> Please review this cleanup in class SafepointMechanism. Since method should_process() is only called by JavaThreads the check in local_poll() is redundant. It also makes the generated code for should_process() unnecessary long since we only need to check the poll bit. I only had to fix a couple of calls from the zero interpreter that where passing a Thread* instead of a JavaThread*.
> Tested tier1 in mach5 and that it builds zero.
>
> Thanks,
> Patricio
Thumbs up.
-------------
Marked as reviewed by dcubed (Reviewer).
PR: https://git.openjdk.java.net/jdk/pull/3603
More information about the hotspot-runtime-dev
mailing list