From b47aa381fddac78b63142b99ea87cfd8455d99e0 Mon Sep 17 00:00:00 2001
From: rtoy <rtoy>
Date: Tue, 9 Sep 2008 15:01:00 +0000
Subject: [PATCH] Update comments.

Config.sun4_solaris_gcc:
o Note that gdb 6.8 handle signals correctly and can be used to debug
  cmucl.

Config.sun4_solaris_sunc:
o Add link to why we can't use inline with Sun Studio 11.
---
 lisp/Config.sun4_solaris_gcc  | 11 ++++++-----
 lisp/Config.sun4_solaris_sunc |  8 +++-----
 2 files changed, 9 insertions(+), 10 deletions(-)

diff --git a/lisp/Config.sun4_solaris_gcc b/lisp/Config.sun4_solaris_gcc
index babd5f990..7d479b84f 100644
--- a/lisp/Config.sun4_solaris_gcc
+++ b/lisp/Config.sun4_solaris_gcc
@@ -38,14 +38,15 @@ endif
 
 CPPFLAGS = -I. -I$(PATH1) -DSOLARIS -DSVR4 $(CC_V8PLUS) $(LINKAGE) $(GENCGC)
 
-# Note: If you want to be able to run gdb with cmucl, the only version
-# of gdb that still works for me (rtoy) on Solaris is 4.18.  Versions
-# 5.x and later (including 6.3) are incapable of passing signals
-# (SIGILL and SIGSEGV) to lisp properly.
+# Note: If you want to be able to use gdb with cmucl, you need to use
+# gdb 4.18 (or earlier?) or use gdb 6.8.  Versions 5.x and later
+# (including 6.3) appear to be incapable of passing signals (SIGILL
+# and SIGSEGV) to lisp properly.
 #
 # But, gcc 3.3.3 and later appear to produce executables that gdb 4.18
 # can't handle.  So, we should probably stick with gcc 3.3.2.  Adjust
-# the CC variable appropriately, if necessary.
+# the CC variable appropriately, if necessary.  But gdb 6.8 handles
+# 3.4.3 just fine.
 
 CC = gcc -O -Wall
 #CC = /apps/gnu/solaris2.8/gcc-3.3.3/bin/gcc -O1 -Wall
diff --git a/lisp/Config.sun4_solaris_sunc b/lisp/Config.sun4_solaris_sunc
index 716b730bd..01e4ba45a 100644
--- a/lisp/Config.sun4_solaris_sunc
+++ b/lisp/Config.sun4_solaris_sunc
@@ -35,19 +35,17 @@ GENCGC = -DGENCGC
 GC_SRC = gencgc.c
 endif
 
+# Sun Studio 11 can't inline the functions in gencgc.c.  See
+# <http://blogs.sun.com/dew/entry/c99_inline_function> for an
+# explanation.  Hence, we disable inline.
 CPPFLAGS = -Dinline= -I. -I$(PATH1) -DSOLARIS -DSVR4 $(CC_V8PLUS) $(LINKAGE) $(GENCGC)
 
 
 CC = cc -O -xlibmieee
 CPP = cc -E
-# -xs so we can use gdb
 CFLAGS = -g $(CC_V8PLUS)
 ASFLAGS = $(AS_V8PLUS)
 
-# SunC 2.0.1 doesn't understand .S suffixes anymore.  But the free Sun
-# Studio 11 does (which is all I (rtoy) have access to anymore, so
-# comment that out.
-#COMPILE.S = as -P $(CPPFLAGS) $(AS_V8PLUS)
 NM = $(PATH1)/solaris-nm
 ASSEM_SRC = sparc-assem.S
 ARCH_SRC = sparc-arch.c
-- 
GitLab