RFR: 8265453: SafepointMechanism::should_process() should receive JavaThread*

Patricio Chilano Mateo pchilanomate at openjdk.java.net
Wed Apr 21 17:52:56 UTC 2021


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

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

Commit messages:
 - v1

Changes: https://git.openjdk.java.net/jdk/pull/3603/files
 Webrev: https://webrevs.openjdk.java.net/?repo=jdk&pr=3603&range=00
  Issue: https://bugs.openjdk.java.net/browse/JDK-8265453
  Stats: 26 lines in 3 files changed: 2 ins; 13 del; 11 mod
  Patch: https://git.openjdk.java.net/jdk/pull/3603.diff
  Fetch: git fetch https://git.openjdk.java.net/jdk pull/3603/head:pull/3603

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


More information about the hotspot-runtime-dev mailing list