# HG changeset patch # User neliasso # Date 1498570605 -7200 # Node ID 18a062f9a2274d4e88b1fe908d980c5c6ec0f97c # Parent 697dad96a3df3c4aaf11efe778571934b81610ac 8181868: Remove use of 'has_fast_fxtof()' Summary: No longer used Reviewed-by: kvn Contributed-by: phedlin@oracle.com diff -r 697dad96a3df -r 18a062f9a227 hotspot/src/cpu/sparc/vm/sparc.ad --- a/hotspot/src/cpu/sparc/vm/sparc.ad Tue Jun 27 15:34:36 2017 +0200 +++ b/hotspot/src/cpu/sparc/vm/sparc.ad Tue Jun 27 15:36:45 2017 +0200 @@ -1760,13 +1760,8 @@ return true; } -/* NOTE: All currently supported SPARC HW provides fast conversion. - * - * TODO: Clean-out code depending on 'has_fast_fxtof'. - */ -const bool Matcher::convL2FSupported(void) { - return VM_Version::has_fast_fxtof(); -} +// NOTE: All currently supported SPARC HW provides fast conversion. +const bool Matcher::convL2FSupported(void) { return true; } // Is this branch offset short enough that a short branch can be used? // @@ -8244,40 +8239,6 @@ ins_pipe(fmulD_reg_reg); %} -instruct convL2D_reg_slow_fxtof(regD dst, stackSlotL src) %{ - match(Set dst (ConvL2D src)); - ins_cost(DEFAULT_COST*8 + MEMORY_REF_COST*6); - - expand %{ - regD_low tmpsrc; - iRegI ix43300000; - iRegI ix41f00000; - stackSlotL lx43300000; - stackSlotL lx41f00000; - regD_low dx43300000; - regD dx41f00000; - regD tmp1; - regD_low tmp2; - regD tmp3; - regD tmp4; - - stkL_to_regD(tmpsrc, src); - - loadConI_x43300000(ix43300000); - loadConI_x41f00000(ix41f00000); - regI_to_stkLHi(lx43300000, ix43300000); - regI_to_stkLHi(lx41f00000, ix41f00000); - stkL_to_regD(dx43300000, lx43300000); - stkL_to_regD(dx41f00000, lx41f00000); - - convI2D_regDHi_regD(tmp1, tmpsrc); - regDHi_regDLo_to_regD(tmp2, dx43300000, tmpsrc); - subD_regD_regD(tmp3, tmp2, dx43300000); - mulD_regD_regD(tmp4, tmp1, dx41f00000); - addD_regD_regD(dst, tmp3, tmp4); - %} -%} - // Long to Double conversion using fast fxtof instruct convL2D_helper(regD dst, regD tmp) %{ effect(DEF dst, USE tmp); @@ -8289,7 +8250,6 @@ %} instruct convL2D_stk_fast_fxtof(regD dst, stackSlotL src) %{ - predicate(VM_Version::has_fast_fxtof()); match(Set dst (ConvL2D src)); ins_cost(DEFAULT_COST + 3 * MEMORY_REF_COST); expand %{ diff -r 697dad96a3df -r 18a062f9a227 hotspot/src/cpu/sparc/vm/vm_version_sparc.cpp --- a/hotspot/src/cpu/sparc/vm/vm_version_sparc.cpp Tue Jun 27 15:34:36 2017 +0200 +++ b/hotspot/src/cpu/sparc/vm/vm_version_sparc.cpp Tue Jun 27 15:36:45 2017 +0200 @@ -207,7 +207,7 @@ char buf[512]; jio_snprintf(buf, sizeof(buf), - "%s%s%s%s%s%s%s%s%s%s%s%s%s%s%s%s%s%s%s%s%s%s%s%s%s%s%s%s%s%s%s%s%s%s%s%s%s%s", + "%s%s%s%s%s%s%s%s%s%s%s%s%s%s%s%s%s%s%s%s%s%s%s%s%s%s%s%s%s%s%s%s%s%s%s%s%s", (has_v9() ? "v9" : ""), (has_popc() ? ", popc" : ""), (has_vis1() ? ", vis1" : ""), @@ -245,7 +245,6 @@ (has_fast_bis() ? ", *bis" : ""), (has_fast_ld() ? ", *ld" : ""), (has_fast_cmove() ? ", *cmove" : ""), - (has_fast_fxtof() ? ", *fxtof" : ""), (has_fast_ind_br() ? ", *ind_br" : ""), (has_blk_zeroing() ? ", *blk_zeroing" : "")); diff -r 697dad96a3df -r 18a062f9a227 hotspot/src/cpu/sparc/vm/vm_version_sparc.hpp --- a/hotspot/src/cpu/sparc/vm/vm_version_sparc.hpp Tue Jun 27 15:34:36 2017 +0200 +++ b/hotspot/src/cpu/sparc/vm/vm_version_sparc.hpp Tue Jun 27 15:36:45 2017 +0200 @@ -252,7 +252,6 @@ static bool has_fast_bis() { return (_features & CPU_fast_bis_msk) != 0; } static bool has_fast_ld() { return (_features & CPU_fast_ld_msk) != 0; } static bool has_fast_cmove() { return (_features & CPU_fast_cmove_msk) != 0; } - static bool has_fast_fxtof() { return true; } // If indirect and direct branching is equally fast. static bool has_fast_ind_br() { return (_features & CPU_fast_ind_br_msk) != 0; }