[commits] r11039 - in /libdfp/trunk: ChangeLog configure.ac printf_dfp.c tests/test-fpclassify.c tests/test-logd.c

[ryanarn@xxxxxxxxxx]

Author: ryanarn
Date: Fri Jul 23 10:59:51 2010
New Revision: 11039

Log:
Removed unnecessary comments and commented-out code.

2010-07-23  Ryan S. Arnold  <rsa@xxxxxxxxxx>

	* tests/test-logd.c: Removed unnecessary comments.
	* tests/test-fpclassify.c: Removed unnecessary comments.
	* configure.ac: Removed unnecessary comments.
	* printf_dfp.c: Removed unnecessary comments.


Modified:
    libdfp/trunk/ChangeLog
    libdfp/trunk/configure.ac
    libdfp/trunk/printf_dfp.c
    libdfp/trunk/tests/test-fpclassify.c
    libdfp/trunk/tests/test-logd.c

Modified: libdfp/trunk/ChangeLog
==============================================================================
--- libdfp/trunk/ChangeLog (original)
+++ libdfp/trunk/ChangeLog Fri Jul 23 10:59:51 2010
@@ -1,3 +1,11 @@
+2010-07-23  Ryan S. Arnold  <rsa@xxxxxxxxxx>
+
+	* tests/test-logd.c: Removed unnecessary comments.
+	* tests/test-fpclassify.c: Removed unnecessary comments.
+	* configure.ac: Removed unnecessary comments.
+	* printf_dfp.c: Removed unnecessary comments.
+
+
 2010-07-23  Ryan S. Arnold  <rsa@xxxxxxxxxx>
 
 	* Makefile.in: Removed commented out code.

Modified: libdfp/trunk/configure.ac
==============================================================================
--- libdfp/trunk/configure.ac (original)
+++ libdfp/trunk/configure.ac Fri Jul 23 10:59:51 2010
@@ -220,14 +220,6 @@
 dnl We need to use [ and ] for other purposes for a while now.
 changequote(,)dnl
 
-# Expand the configuration machine name into a subdirectory by architecture
-# type and particular chip.
-#cc_msize=
-#	powerpc) base_machine=powerpc machine=powerpc/powerpc32 cc_msize=32 ;;
-#	powerpc64) base_machine=powerpc machine=powerpc/powerpc64 cc_msize=64;;
-#	s390) base_machine=s390 machine=s390/s390-32 cc_msize=31;;
-#	s390x) base_machine=s390 machine=s390/s390-64 cc_msize=64;;
-
 case "$machine" in
 	powerpc) base_machine=powerpc machine=powerpc/powerpc32 ;;
 	powerpc64) base_machine=powerpc machine=powerpc/powerpc64 ;;
@@ -246,20 +238,10 @@
 # export the computed default cc_msize to the Makefile.in.
 AC_PROG_CC(gcc)
 
-# Automatically exported by AC_PROG_CC
-#AC_SUBST(CC)
-#AC_SUBST(CFLAGS)
-
 OBJDUMP="$OBJDUMP"
 AS="$AS"
 AC_SUBST(OBJDUMP)
 AC_SUBST(AS)
-
-# It was considered to add this to CFLAGS before AC_PROG_CC was declared to
-# make sure that libdecnumber got all of the flags but it was decided to
-# simply invoke decnumber's `make' with an exported and overridden CFLAGS.
-#AC_SUBST(cc_mzarch)
-#AC_SUBST(cc_msize)
 
 AC_SUBST(CFLAGS)
 AC_SUBST(ASFLAGS)
@@ -311,7 +293,6 @@
 # Check for GLIBC 2.10 headers
 have_glibc_2_10_headers=yes
 saved_CFLAGS="$CFLAGS"
-#CFLAGS="-m$cc_msize $CFLAGS $glibc_headers_inc"
 CFLAGS="$CFLAGS $glibc_headers_inc"
 
 AC_MSG_CHECKING([for printf-hook register_printf_type() in printf.h to verify GLIBC 2.10])
@@ -370,7 +351,6 @@
 # Check for GLIBC 2.10 runtime support
 have_glibc_2_10=yes
 saved_CFLAGS="$CFLAGS"
-#CFLAGS="-m$cc_msize $CFLAGS $glibc_headers_inc"
 CFLAGS="$CFLAGS $glibc_headers_inc"
 saved_LIBS="$LIBS"
 LIBS="$LIBS $glibc_build_lib"

Modified: libdfp/trunk/printf_dfp.c
==============================================================================
--- libdfp/trunk/printf_dfp.c (original)
+++ libdfp/trunk/printf_dfp.c Fri Jul 23 10:59:51 2010
@@ -636,7 +636,9 @@
   if (prec < 0)
     prec = default_prec;
 
-  /* do rounding if precision is less than the decimal type */
+  /* Do rounding if precision is less than the decimal type.  On hardware DFP
+   * this could probably easily be done with quantize but on soft-dfp the
+   * existing method would be faster.  */
   if (prec < input_prec)
     {
       int index, roundmode = 0;

Modified: libdfp/trunk/tests/test-fpclassify.c
==============================================================================
--- libdfp/trunk/tests/test-fpclassify.c (original)
+++ libdfp/trunk/tests/test-fpclassify.c Fri Jul 23 10:59:51 2010
@@ -96,14 +96,10 @@
   {__LINE__, 0.0DF, FP_ZERO, "%d"},
   {__LINE__, 2.3DF, FP_NORMAL, "%d"},
   {__LINE__, -2.3DF, FP_NORMAL, "%d"},
-/* Known failure on hard-dfp due to promotion to _Decimal64.  These are well
- * within the range of NORMAL for _Decimal64.  */
-//#ifndef _ARCH_PWR6 
   {__LINE__, __DEC32_SUBNORMAL_MIN__, FP_SUBNORMAL, "%d"}, /* Subnormal  */
   {__LINE__, -__DEC32_SUBNORMAL_MIN__, FP_SUBNORMAL, "%d"}, /* Subnormal  */
   {__LINE__, 0.000001E-95DF, FP_SUBNORMAL, "%d"},
   {__LINE__, -0.000001E-95DF, FP_SUBNORMAL, "%d"},
-//#endif
   {__LINE__, __DEC32_MIN__, FP_NORMAL, "%d"},
   {0,0,0,0 }
 };

Modified: libdfp/trunk/tests/test-logd.c
==============================================================================
--- libdfp/trunk/tests/test-logd.c (original)
+++ libdfp/trunk/tests/test-logd.c Fri Jul 23 10:59:51 2010
@@ -74,7 +74,6 @@
   /* DEC_NAN is +0,000,000,000,000,000E-398 so test against that
    * since you can't compare DEC_NAN to DEC_NAN.  */
   {__LINE__, -1.0DD, DECLET64_NAN},
-  //{__LINE__, -1.0DD, "-0,000,000,000,000,010E-1"},
 #ifdef _ARCH_PWR6 /* This returns NaN in the hard-DFP case.  */
   {__LINE__, __DEC64_MAX__, DECLET64_NAN},
 #endif