RFR: 8341831: PhaseCFG::insert_anti_dependences asserts with "no loads" [v2]

Vladimir Kozlov kvn at openjdk.org
Thu Oct 10 17:00:17 UTC 2024


On Thu, 10 Oct 2024 15:37:22 GMT, Igor Veresov <iveresov at openjdk.org> wrote:

>> `CacheWB` nodes are peculiar in a sense that they both are anti-dependent and produce memory. I think it's reasonable to relax the assert in `insert_anti_dependences()` to work around their properties.
>
> Igor Veresov has updated the pull request with a new target base due to a merge or a rebase. The pull request now contains two commits:
> 
>  - Remove the test from the problem list
>  - 8341831: PhaseCFG::insert_anti_dependences asserts with "no loads"
>    Summary: Relax assert to deal with CacheWB nodes

Please add comment into code explaining change.

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

PR Review: https://git.openjdk.org/jdk/pull/21455#pullrequestreview-2360944637


More information about the hotspot-compiler-dev mailing list