Tcl

if there's something still to be done for the
8.5.8 and 8.6b2 releases, I've lost track of it.

If there is something, please open a new ticket
on it so we can dig it out of the clutter.

I claimed this one since we have a few old
Sun's and SGI's around for testing. However,
this item has moved on since it was reported,
and now deals with systems I do not have.

I still see the test failures on IRIX64 6.5 with
the MIPSpro 7.4.4m compiler, but that platform
combo is probably too old to be worth worrying
about.

To get around the libsunmath dependecy mentioned by tclguy one can use the
libm based interface documented on x86 via the C99 fenv.h header and just
libm.
http://docs.sun.com/source/819-3693/ncg_lib.html

#include <fenv.h>

fesetprec(FE_DBLPREC); is like
ieee_flags("set","precision","double",NULL);

fesetprec(FE_LDBLPREC) is like the ieee_flags("clear"...) stuff.

Suggest that gsyoungblood's patch incorrectly builds 64 bit objects on 32
bit boxes. (The patch against the original code is correct; the original
code was incorrect.)

The patch should be as follows. Note that at configure:8417,
configure:8651, tcl.m4:1916, tcl.m4:1978 we exclude i386 boxes; the
original patch included them. (We don't want -m64 in CFLAGS on a 32-bit
box.)

Regards,
Geoff.
first six letters on the top row (i.e. q...) of a US / UK keyboard at
acm.org.


diff -bur tcl8.5.6/unix/configure tcl8.5.6-patched/unix/configure
--- tcl8.5.6/unix/configure 2008-12-22 14:16:08.000000000 +0000
+++ tcl8.5.6-patched/unix/configure 2009-01-31 16:19:42.495469123 +0000
@@ -8417,7 +8417,7 @@


else
- if test "$arch" = "amd64 i386"; then
+ if test "$arch" = "amd64"; then

if test "$GCC" = yes; then

@@ -8468,7 +8468,7 @@
arch=`isainfo`
echo "$as_me:$LINENO: checking whether to use -lsunmath for fp rounding
control" >&5
echo $ECHO_N "checking whether to use -lsunmath for fp rounding
control... $ECHO_C" >&6
- if test "$arch" = "amd64 i386"; then
+ if test "$arch" = "amd64" -o "$arch" = "i386"; then

echo "$as_me:$LINENO: result: yes" >&5
echo "${ECHO_T}yes" >&6
@@ -8651,7 +8651,7 @@
#CC_SEARCH_FLAGS="${CC_SEARCH_FLAGS},-R,$v9gcclibdir"

else
- if test "$arch" = "amd64 i386"; then
+ if test "$arch" = "amd64"; then

SHLIB_LD="$SHLIB_LD -m64 -static-libgcc"

diff -bur tcl8.5.6/unix/tcl.m4 tcl8.5.6-patched/unix/tcl.m4
--- tcl8.5.6/unix/tcl.m4 2008-12-22 14:16:08.000000000 +0000
+++ tcl8.5.6-patched/unix/tcl.m4 2009-01-31 16:20:05.425468467 +0000
@@ -1916,7 +1916,7 @@
# Solaris 64 uses this as well
#LD_LIBRARY_PATH_VAR="LD_LIBRARY_PATH_64"
])
- ], [AS_IF([test "$arch" = "amd64 i386"], [
+ ], [AS_IF([test "$arch" = "amd64"], [
AS_IF([test "$GCC" = yes], [
case $system in
SunOS-5.1[[1-9]]*|SunOS-5.[[2-9]][[0-9]]*)
@@ -1947,7 +1947,7 @@
AS_IF([test "$GCC" = yes],[use_sunmath=no],[
arch=`isainfo`
AC_MSG_CHECKING([whether to use -lsunmath for fp rounding control])
- AS_IF([test "$arch" = "amd64 i386"], [
+ AS_IF([test "$arch" = "amd64" -o "$arch" = "i386"], [
AC_MSG_RESULT([yes])
MATH_LIBS="-lsunmath $MATH_LIBS"
AC_CHECK_HEADER(sunmath.h)
@@ -1978,7 +1978,7 @@
# path, remove so name and append 'sparcv9'
#v9gcclibdir="`gcc -print-file-name=libgcc_s.so` | ..."
#CC_SEARCH_FLAGS="${CC_SEARCH_FLAGS},-R,$v9gcclibdir"
- ], [AS_IF([test "$arch" = "amd64 i386"], [
+ ], [AS_IF([test "$arch" = "amd64"], [
SHLIB_LD="$SHLIB_LD -m64 -static-libgcc"
])])
])

Ran into what I believe is a related problem when attempting to compile
using Sun Studio Express 2008-11 on Solaris 10u6 32-bit Intel.

Compiling would fail with undefined symbol ieee_flags in libtcl8.5.so.
Further digging revealed that the check whether or not to use -lsunmath for
fp rounding control was returning negative, causing -lsunmath not to be
added to MATH_LIBS. The problem appeared to be the use of a list (amd64
i386) when testing arch. Changing it to test each arch individually
appeared to resolve the problem. Assuming this is correct, there's a
similar test for sparc (sparcv9 sparc) that may need tweaking too.

The following patch allows it to compile:
diff -bur tcl8.5.6/unix/configure tcl8.5.6-solaris/unix/configure
--- tcl8.5.6/unix/configure Mon Dec 22 07:16:08 2008
+++ tcl8.5.6-solaris/unix/configure Mon Jan 5 14:36:32 2009
@@ -8417,7 +8417,7 @@


else
- if test "$arch" = "amd64 i386"; then
+ if test "$arch" = "amd64" -o "$arch" = "i386" ; then

if test "$GCC" = yes; then

@@ -8468,7 +8468,7 @@
arch=`isainfo`
echo "$as_me:$LINENO: checking whether to use -lsunmath
for fp rounding control" >&5
echo $ECHO_N "checking whether to use -lsunmath for fp rounding
control... $ECHO_C" >&6
- if test "$arch" = "amd64 i386"; then
+ if test "$arch" = "amd64" -o "$arch" = "i386" ; then

echo "$as_me:$LINENO: result: yes" >&5
echo "${ECHO_T}yes" >&6
@@ -8651,7 +8651,7 @@

#CC_SEARCH_FLAGS="${CC_SEARCH_FLAGS},-R,$v9gcclibdir"

else
- if test "$arch" = "amd64 i386"; then
+ if test "$arch" = "amd64" -o "$arch" = "i386" ; then

SHLIB_LD="$SHLIB_LD -m64 -static-libgcc"

diff -bur tcl8.5.6/unix/tcl.m4 tcl8.5.6-solaris/unix/tcl.m4
--- tcl8.5.6/unix/tcl.m4 Mon Dec 22 07:16:08 2008
+++ tcl8.5.6-solaris/unix/tcl.m4 Mon Jan 5 14:35:08 2009
@@ -1916,7 +1916,7 @@
# Solaris 64 uses this as well
#LD_LIBRARY_PATH_VAR="LD_LIBRARY_PATH_64"
])
- ], [AS_IF([test "$arch" = "amd64 i386"], [
+ ], [AS_IF([test "$arch" = "amd64" -o "$arch" = "i386"], [
AS_IF([test "$GCC" = yes], [
case $system in
SunOS-5.1[[1-9]]*|SunOS-5.[[2-9]][[0-9]]*)
@@ -1947,7 +1947,7 @@
AS_IF([test "$GCC" = yes],[use_sunmath=no],[
arch=`isainfo`
AC_MSG_CHECKING([whether to use -lsunmath for fp rounding
control])
- AS_IF([test "$arch" = "amd64 i386"], [
+ AS_IF([test "$arch" = "amd64" -o "$arch" = "i386"], [
AC_MSG_RESULT([yes])
MATH_LIBS="-lsunmath $MATH_LIBS"
AC_CHECK_HEADER(sunmath.h)
@@ -1978,7 +1978,7 @@
# path, remove so name and append 'sparcv9'
#v9gcclibdir="`gcc -print-file-name=libgcc_s.so` |
..."

#CC_SEARCH_FLAGS="${CC_SEARCH_FLAGS},-R,$v9gcclibdir"
- ], [AS_IF([test "$arch" = "amd64 i386"], [
+ ], [AS_IF([test "$arch" = "amd64" -o "$arch" =
"i386"], [
SHLIB_LD="$SHLIB_LD -m64 -static-libgcc"
])])
])

We have recently discovered another issue with the sunmath fix for
solaris-x86 - libsunmath only comes with Sun's Forte and isn't standard on
default solaris-x86 machines. We are currently having to work around this.

Had the chance to do some testing
on an IRIX64 6.5 system with the
MIPSpro 7.4.4m compiler, and it
exhibits similar troubles in the
form of 23 failing tests in binary.test
and expr.test, mostly differences
in the LSB in underflow scenarios.

Patch applied to head and core-8-5-branch.

Verified your updated patch with my x86 VM and the sun CC, works fine and
as expected.

I have updated the attached patch to the Tcl 8.5 head, should work for Tcl
head (8.6) as well.
In tcl.m4 mostly location differences, and making one rejected chunk
viable to due to conflict with other changes made in the meantime.

In tclStrToD.c I had to add "&& !defined(__GNUC__)" to the ifdeffery of
three uses of ieee_flags. These missing exposed ieee_flags to a gcc build
on a solaris 2.10 amd64 box, causing link failure.

Michael, please check that the updated patch still works for you.

File Added: sunmath-updated.diff

I don't have a Solaris-x86 to test on, but the patch looks right.
Jeff, could I trouble you or Andreas to apply the patch,
make test, and commit if everything works? Thanks.

The errors in expr dissapear when linking with sunmath (which requires -z
textoff to the linker, otherwise LD barfs) and applying a small fix to
tclStrToD.c.

The attached patch works on opensolaris. Would be good if someone with a
regular solaris setup on x86 and sparc could check if the patch doesn't
break something.
File Added: sunmath.diff