[13] RFR (S): 8217918: C2: -XX:+AggressiveUnboxing is broken

Vladimir Kozlov vladimir.kozlov at oracle.com
Fri Feb 1 05:57:13 UTC 2019

Nice! Did you consider to update indexes in separate pass at the end of PhaseRenumberLive? Would it be simpler?

Vladimir K

On 1/31/19 11:10 AM, Vladimir Ivanov wrote:
> https://bugs.openjdk.java.net/browse/JDK-8217918
> http://cr.openjdk.java.net/~vlivanov/8217918/webrev.00/
> When -XX:+AggressiveUnboxing is enabled, LoadNode::split_through_phi() produces Phi nodes with non-negative _inst_mem_id 
> & _inst_id early enough, so it breaks PhaseRenumber pass which doesn't support nodes with embedded IDs.
> Proposed fix tracks nodes with embedded IDs and updates them once renumbering pass over the graph is over.
> Testing: hs-precheckin-comp, tier1-5
> Best regards,
> Vladimir Ivanov

More information about the hotspot-compiler-dev mailing list