RFR: 8366984: Remove delay slot support

Emanuel Peter epeter at openjdk.org
Tue Sep 9 09:14:05 UTC 2025


On Fri, 5 Sep 2025 14:24:50 GMT, Daniel Jeliński <djelinski at openjdk.org> wrote:

> SPARC was the only supported architecture that uses a delay slot. The SPARC port was removed in JDK 15, and the code is effectively dead. Let's remove it.
> 
> The changes are no-op on all architectures that do not use delay slots. I still tested tier 1-5 on mach5, no related failures.

Looks reasonable, thanks for doing the cleanup! I have 2 minor questions though.
(please also run additional stress testing, see slack)

src/hotspot/cpu/arm/arm.ad line 3383:

> 3381:     BR     : R;
> 3382: %}
> 3383: 

Where was this used? Or is it an unrelated cleanup?

src/hotspot/share/adlc/adlparse.cpp line 1394:

> 1392:           parse_err(SYNERR, "Using obsolete token, branch_has_delay_slot");
> 1393:           break;
> 1394:         }

I'm curious: why do you add that special warning? It would fail later anyway, right? Are we expecting anyone to parse things produced by different versions?

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

Marked as reviewed by epeter (Reviewer).

PR Review: https://git.openjdk.org/jdk/pull/27119#pullrequestreview-3200246647
PR Review Comment: https://git.openjdk.org/jdk/pull/27119#discussion_r2332626258
PR Review Comment: https://git.openjdk.org/jdk/pull/27119#discussion_r2332620923


More information about the hotspot-dev mailing list