RFR: 8257630: ReplacedNodes doesn't handle non-CFG multi nodes
Nils Eliasson
neliasso at openjdk.java.net
Wed Dec 2 22:22:55 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
Looks good.
-------------
Marked as reviewed by neliasso (Reviewer).
PR: https://git.openjdk.java.net/jdk/pull/1580
More information about the hotspot-compiler-dev
mailing list