stan/jdk-24
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

8203215: IdealLoopTree::split_outer_loop leaves phi-nodes with only one input

Browse Source 
Make sure the phis are transformed

Reviewed-by: roland, kvn, rbackman
This commit is contained in:
Nils Eliasson 2018-05-17 12:08:32 +02:00
parent c2b147225c
commit fbbbc35d32
1 changed files with 8 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

11
src/hotspot/share/opto/loopnode.cpp
View File

@ -2641,9 +2641,6 @@ void PhaseIdealLoop::build_and_optimize(bool do_split_ifs, bool skip_loop_opts)
int old_progress = C->major_progress();
uint orig_worklist_size = _igvn._worklist.size();
// Reset major-progress flag for the driver's heuristics
C->clear_major_progress();
#ifndef PRODUCT
// Capture for later assert
uint unique = C->unique();
@ -2714,11 +2711,16 @@ void PhaseIdealLoop::build_and_optimize(bool do_split_ifs, bool skip_loop_opts)
if( !_verify_me && !_verify_only && _ltree_root->_child ) {
C->print_method(PHASE_BEFORE_BEAUTIFY_LOOPS, 3);
if( _ltree_root->_child->beautify_loops( this ) ) {
// IdealLoopTree::split_outer_loop may produce phi-nodes with a single in edge.
// Transform them away.
_igvn.optimize();
// Re-build loop tree!
_ltree_root->_child = NULL;
_nodes.clear();
reallocate_preorders();
build_loop_tree();
// Check for bailout, and return
if (C->failing()) {
return;
@ -2730,6 +2732,9 @@ void PhaseIdealLoop::build_and_optimize(bool do_split_ifs, bool skip_loop_opts)
}
}
// Reset major-progress flag for the driver's heuristics
C->clear_major_progress();
// Build Dominators for elision of NULL checks & loop finding.
// Since nodes do not have a slot for immediate dominator, make
// a persistent side array for that info indexed on node->_idx.