RFR: 8265784: [C2] Hoisting of DecodeN leaves MachTemp inputs behind [v2]
Martin Doerr
mdoerr at openjdk.java.net
Mon May 3 14:04:22 UTC 2021
> 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.
-------------
Changes:
- all: https://git.openjdk.java.net/jdk/pull/3637/files
- new: https://git.openjdk.java.net/jdk/pull/3637/files/86a4a6a8..0c893cd4
Webrevs:
- full: https://webrevs.openjdk.java.net/?repo=jdk&pr=3637&range=01
- incr: https://webrevs.openjdk.java.net/?repo=jdk&pr=3637&range=00-01
Stats: 1 line in 1 file changed: 1 ins; 0 del; 0 mod
Patch: https://git.openjdk.java.net/jdk/pull/3637.diff
Fetch: git fetch https://git.openjdk.java.net/jdk pull/3637/head:pull/3637
PR: https://git.openjdk.java.net/jdk/pull/3637
More information about the hotspot-compiler-dev
mailing list