OpenJDK / jdk / jdk
changeset 56159:2172fd713350
8230428: Cleanup dead CastIP node code in formssel.cpp
Summary: Removed dead code.
Reviewed-by: roland
author | thartmann |
---|---|
date | Tue, 03 Sep 2019 10:03:13 +0200 |
parents | f571cc16478a |
children | 197f36af94f3 |
files | src/hotspot/share/adlc/formssel.cpp |
diffstat | 1 files changed, 3 insertions(+), 30 deletions(-) [+] |
line wrap: on
line diff
--- a/src/hotspot/share/adlc/formssel.cpp Mon Sep 02 15:59:37 2019 +0100 +++ b/src/hotspot/share/adlc/formssel.cpp Tue Sep 03 10:03:13 2019 +0200 @@ -4007,40 +4007,13 @@ } int MatchRule::is_ideal_copy() const { - if( _rChild ) { - const char *opType = _rChild->_opType; -#if 1 - if( strcmp(opType,"CastIP")==0 ) - return 1; -#else - if( strcmp(opType,"CastII")==0 ) - return 1; - // Do not treat *CastPP this way, because it - // may transfer a raw pointer to an oop. - // If the register allocator were to coalesce this - // into a single LRG, the GC maps would be incorrect. - //if( strcmp(opType,"CastPP")==0 ) - // return 1; - //if( strcmp(opType,"CheckCastPP")==0 ) - // return 1; - // - // Do not treat CastX2P or CastP2X this way, because - // raw pointers and int types are treated differently - // when saving local & stack info for safepoints in - // Output(). - //if( strcmp(opType,"CastX2P")==0 ) - // return 1; - //if( strcmp(opType,"CastP2X")==0 ) - // return 1; -#endif + if (is_chain_rule(_AD.globalNames()) && + _lChild && strncmp(_lChild->_opType, "stackSlot", 9) == 0) { + return 1; } - if( is_chain_rule(_AD.globalNames()) && - _lChild && strncmp(_lChild->_opType,"stackSlot",9)==0 ) - return 1; return 0; } - int MatchRule::is_expensive() const { if( _rChild ) { const char *opType = _rChild->_opType;