Integrated: 8257630: C2: ReplacedNodes doesn't handle non-CFG multi nodes

Vladimir Ivanov vlivanov at openjdk.java.net
Fri Dec 4 07:56:58 UTC 2020


On Wed, 2 Dec 2020 21:21:19 GMT, Vladimir Ivanov <vlivanov at openjdk.org> wrote:

> When looking up an immediate CFG node, ReplacedNodes assumes that control input is always a CFG node. 
> It's not always the case: for example, a projection on a non-CFG multi node (`SCMemProj` and `LoadStore` respectively).
> 
> Proposed fix is to skip projection nodes first before checking for CFG.
> 
> Testing:
> - [x] hs-tier1-6 w/ -XX:+AlwaysIncrementalInlining

This pull request has now been integrated.

Changeset: 4390f2c8
Author:    Vladimir Ivanov <vlivanov at openjdk.org>
URL:       https://git.openjdk.java.net/jdk/commit/4390f2c8
Stats:     8 lines in 1 file changed: 3 ins; 1 del; 4 mod

8257630: C2: ReplacedNodes doesn't handle non-CFG multi nodes

Reviewed-by: neliasso, kvn, thartmann

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

PR: https://git.openjdk.java.net/jdk/pull/1580


More information about the hotspot-compiler-dev mailing list