JDK-8229015 : Check removal of yank() in PhaseIdealLoop::split_if_with_blocks_post
  • Type: Enhancement
  • Component: hotspot
  • Sub-Component: compiler
  • Affected Version: 14
  • Priority: P4
  • Status: Closed
  • Resolution: Duplicate
  • Submitted: 2019-08-02
  • Updated: 2022-02-07
  • Resolved: 2022-02-07
The Version table provides details related to the release that this issue/RFE will be addressed.

Unresolved : Release in which this issue/RFE will be addressed.
Resolved: Release in which this issue/RFE has been resolved.
Fixed : Release in which this issue/RFE has been fixed. The release containing this fix may be available for download as an Early Access Release or a General Availability Release.

To download the current JDK release, click here.
Other
tbdResolved
Related Reports
Duplicate :  
JDK-8252372 - Check if cloning is required to move loads out of loops in PhaseIdealLoop::split_if_with_blocks_post()
Relates :  
JDK-6394013 - C2: Remove VerifyOpto
Description
In the process of removing the VerifyOpto flag (JDK-6394013) it was unclear if the following line should be removed in PhaseIdealLoop::split_if_with_blocks_post() in looptopts.cpp:

1437           if (!x->is_Load() && !x->is_DecodeNarrowPtr()) _igvn._worklist.yank(x);

This needs some further investigation how a removal affects optimization.
Comments
Was fixed with new implementation to sink nodes out of a loop in JDK-8252372.
07-02-2022