[lworld] RFR: 8250511: [lworld] C2 compilation crashes in PhaseIdealLoop::spinup

Thu Aug 20 10:40:24 UTC 2020

On Thu, 20 Aug 2020 09:42:58 GMT, Roland Westrelin <roland at openjdk.org> wrote:

> In PhaseIdealLoop::flatten_array_element_type_check(), when pushing:
> 
> (LoadKlass (AddP (CastPP ...
> 
> through a Phi, the control of CastPP clone is set to region->in(i) but
> the AddP clone control is set to get_ctrl(phi->in(i)) which can be
> above region->in(i) while the AddP is below the CastPP. Fix is to use
> region->in(i) for all clones.

Looks good to me.

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

Marked as reviewed by thartmann (Committer).

PR: https://git.openjdk.java.net/valhalla/pull/157