# HG changeset patch # User never # Date 1301610612 25200 # Node ID ede3202e00e044ff3fa4a6efcee1e08d34751270 # Parent eed0ba1d011b0a4027b008cbf918898f7ff13589# Parent 9e2b81253ef909ce2f1a67b458305fe0da3aa50a Merge diff -r eed0ba1d011b -r ede3202e00e0 hotspot/src/share/vm/opto/loopTransform.cpp --- a/hotspot/src/share/vm/opto/loopTransform.cpp Thu Mar 31 14:00:41 2011 -0700 +++ b/hotspot/src/share/vm/opto/loopTransform.cpp Thu Mar 31 15:30:12 2011 -0700 @@ -432,6 +432,12 @@ return false; } + // Currently we don't have policy to optimize one iteration loops. + // Maximally unrolling transformation is used for that: + // it is peeled and the original loop become non reachable (dead). + if (trip_count == 1) + return true; + // Do not unroll a loop with String intrinsics code. // String intrinsics are large and have loops. for (uint k = 0; k < _body.size(); k++) { diff -r eed0ba1d011b -r ede3202e00e0 hotspot/src/share/vm/opto/memnode.cpp --- a/hotspot/src/share/vm/opto/memnode.cpp Thu Mar 31 14:00:41 2011 -0700 +++ b/hotspot/src/share/vm/opto/memnode.cpp Thu Mar 31 15:30:12 2011 -0700 @@ -2628,12 +2628,16 @@ Node *StrIntrinsicNode::Ideal(PhaseGVN *phase, bool can_reshape) { if (remove_dead_region(phase, can_reshape)) return this; - Node* mem = phase->transform(in(MemNode::Memory)); - // If transformed to a MergeMem, get the desired slice - uint alias_idx = phase->C->get_alias_index(adr_type()); - mem = mem->is_MergeMem() ? mem->as_MergeMem()->memory_at(alias_idx) : mem; - if (mem != in(MemNode::Memory)) - set_req(MemNode::Memory, mem); + if (can_reshape) { + Node* mem = phase->transform(in(MemNode::Memory)); + // If transformed to a MergeMem, get the desired slice + uint alias_idx = phase->C->get_alias_index(adr_type()); + mem = mem->is_MergeMem() ? mem->as_MergeMem()->memory_at(alias_idx) : mem; + if (mem != in(MemNode::Memory)) { + set_req(MemNode::Memory, mem); + return this; + } + } return NULL; }