RFR: 8280283: Dead compiler code found during the JDK-8272058 code review [v2]
Anton Seoane Ampudia
aseoane at openjdk.org
Mon Jan 12 14:50:38 UTC 2026
On Fri, 5 Dec 2025 10:08:33 GMT, Anton Seoane Ampudia <aseoane at openjdk.org> wrote:
>> This PR removes some dead code that was found during review for [JDK-8272058](https://bugs.openjdk.org/browse/JDK-8272058).
>>
>> `target_addr_for_insn_or_null` is never run with a `ldrw` to `zr` (i.e. a safepoint poll). This is just a remnant from global safepointing, before we moved to using thread-local handshakes. No safepoint polling code reaches this function. More information can be read in the [original code review](https://github.com/openjdk/jdk18/pull/51#discussion_r774922087). Additionally, I have run tiers 1-6 to make sure this path did not exercise.
>>
>> This changeset also cleans up the unused `is_nop` function, following the comments in the issue. Other dead code mentioned there has since been long disappered.
>>
>> **Testing:** passes tiers 1-4
>
> Anton Seoane Ampudia has updated the pull request incrementally with one additional commit since the last revision:
>
> Delete more unused code
Any reviews, please? (@theRealAph maybe?)
Any reviews, please? (@theRealAph maybe?) -- [my last comment got ignored by the bridgebot]
-------------
PR Comment: https://git.openjdk.org/jdk/pull/28473#issuecomment-3660873945
PR Comment: https://git.openjdk.org/jdk/pull/28473#issuecomment-3738906326
More information about the hotspot-compiler-dev
mailing list