RFR: 8352418: Add verification code to check that the associated loop nodes of useless Template Assertion Predicates are dead

Roland Westrelin roland at openjdk.org
Tue Apr 1 12:50:14 UTC 2025


On Mon, 31 Mar 2025 12:28:59 GMT, Christian Hagedorn <chagedorn at openjdk.org> wrote:

> As already suggested in https://github.com/openjdk/jdk/pull/23823, I want to do the following additional verification:
> 
> After `eliminate_useless_predicates()` all now useless `OpaqueTemplateAssertionPredicate` nodes should not have any references to `CountedLoop` nodes that are still in the graph (otherwise, they would have been marked useful). This verification did not work reliably without the full Assertion Predicates fix [JDK-8350577](https://bugs.openjdk.org/browse/JDK-8350577). Since JDK-8350577 is now integrated, I propose to add this additional verification code.
> 
> Thanks,
> Christian

Looks good to me.

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

Marked as reviewed by roland (Reviewer).

PR Review: https://git.openjdk.org/jdk/pull/24326#pullrequestreview-2732570055


More information about the hotspot-compiler-dev mailing list