RFR: 8321278: C2: Partial peeling fails with assert "last_peel <- first_not_peeled" [v2]
Roland Westrelin
roland at openjdk.org
Fri Mar 22 16:37:49 UTC 2024
> The assert fails because peeling happens at a single entry
> `Region`. That `Region` only has a single input because other inputs
> were found unreachable and removed by
> `PhaseIdealLoop::Dominators()`. The fix I propose is to have
> `PhaseIdealLoop::Dominators()` remove the `Region` and its `Phi`s
> entirely in this case.
Roland Westrelin has updated the pull request incrementally with one additional commit since the last revision:
Update test/hotspot/jtreg/compiler/loopopts/TestPartialPeelingAtSingleInputRegion.java
Co-authored-by: Christian Hagedorn <christian.hagedorn at oracle.com>
-------------
Changes:
- all: https://git.openjdk.org/jdk/pull/18353/files
- new: https://git.openjdk.org/jdk/pull/18353/files/5cbab303..bb2cb9ea
Webrevs:
- full: https://webrevs.openjdk.org/?repo=jdk&pr=18353&range=01
- incr: https://webrevs.openjdk.org/?repo=jdk&pr=18353&range=00-01
Stats: 1 line in 1 file changed: 0 ins; 0 del; 1 mod
Patch: https://git.openjdk.org/jdk/pull/18353.diff
Fetch: git fetch https://git.openjdk.org/jdk.git pull/18353/head:pull/18353
PR: https://git.openjdk.org/jdk/pull/18353
More information about the hotspot-compiler-dev
mailing list