8181868: Remove use of 'has_fast_fxtof()'
authorneliasso
Tue, 27 Jun 2017 15:36:45 +0200
changeset 46595 18a062f9a227
parent 46594 697dad96a3df
child 46596 a7c9706d25a9
8181868: Remove use of 'has_fast_fxtof()' Summary: No longer used Reviewed-by: kvn Contributed-by: phedlin@oracle.com
hotspot/src/cpu/sparc/vm/sparc.ad
hotspot/src/cpu/sparc/vm/vm_version_sparc.cpp
hotspot/src/cpu/sparc/vm/vm_version_sparc.hpp
--- 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 %{
--- 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" : ""));
 
--- 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; }