RFR: 8265784: [C2] Hoisting of DecodeN leaves MachTemp inputs behind [v2]
Goetz Lindenmaier
goetz at openjdk.java.net
Mon May 3 14:11:52 UTC 2021
On Mon, 3 May 2021 14:04:22 GMT, Martin Doerr <mdoerr at openjdk.org> wrote:
>> PPC64 and s390 have DecodeN implementations which use a MachTemp input. When LCM hoists the DecodeN, the MachTemp nodes reside in the old block, but should get hoisted together with the DecodeN node.
>> Same is true for load Base input which exists on s390 for example. Unfortunately, that's just a platform specific MachNode which is not nicely recognizable in LCM.
>
> Martin Doerr has updated the pull request incrementally with one additional commit since the last revision:
>
> Add temp node placement assertion.
LGTM
-------------
Marked as reviewed by goetz (Reviewer).
PR: https://git.openjdk.java.net/jdk/pull/3637
More information about the hotspot-compiler-dev
mailing list