[Git][cmucl/cmucl][issue-425-correctly-rounded-math-functions] 2 commits: Revert "Let's try gcc with -march=haswell"

18 Dec 2025


      Raymond Toy pushed to branch issue-425-correctly-rounded-math-functions at cmucl / cmucl


Commits:
ad844ddd by Raymond Toy at 2025-12-18T06:36:11-08:00
Revert "Let's try gcc with -march=haswell"

This reverts commit 07fd3bac2529867a46d9dd610ca4dfaedeb20172.

- - - - -
467e0824 by Raymond Toy at 2025-12-18T07:56:25-08:00
Update value for (acosh 4d0) for core-math

Core-math returns 2.0634370688955603d0.
I'm going to assume that's correct even though (acosh 4w0) is
2.06343706889556054672728117262013w0

And Maxima says:
2.063437068895560546727281172620131871456591449883392499836032693b0

- - - - -


3 changed files:

- .gitlab-ci.yml
- src/lisp/Config.x86_linux
- tests/fdlibm.lisp


Changes:

=====================================
.gitlab-ci.yml
=====================================
@@ -147,7 +147,7 @@ linux:build:
     - job: linux:install
       artifacts: true
   variables:
-    CONFIG: "x86_linux"
+    CONFIG: "x86_linux_clang"
   # These rules is needed so that the static analyzer job can run on a
   # schedule because this is a prerequisite of the analyzer build.  A
   # regular push or merge request does the normal stuff.
@@ -171,7 +171,7 @@ linux:cross-build:
       - linux-4/lisp
   variables:
     # This must match the config used for the linux build!
-    CONFIG: "x86_linux"
+    CONFIG: "x86_linux_clang"
 
   needs:
 
@@ -346,7 +346,7 @@ ubuntu:build:
     - job: ubuntu:install
       artifacts: true
   variables:
-    CONFIG: "x86_linux"
+    CONFIG: "x86_linux_clang"
 
 ubuntu:test:
   <<: *unit_test_configuration


=====================================
src/lisp/Config.x86_linux
=====================================
@@ -6,8 +6,7 @@ CORE_MATH = -DCORE_MATH_SUPPORT_ERRNO
 
 CFLAGS += $(COPT)
 CPPFLAGS += -m32
-#CFLAGS += -rdynamic  -march=pentium4 -mfpmath=sse -mtune=generic
-CFLAGS += -rdynamic  -march=haswell -mfpmath=sse
+CFLAGS += -rdynamic  -march=pentium4 -mfpmath=sse -mtune=generic
 # ANALYZER flag should be set to -fanalyzer when doing the CI builds
 # for the C static analyzer.
 CFLAGS += -D_LARGEFILE_SOURCE -D_FILE_OFFSET_BITS=64 $(ANALYZER)


=====================================
tests/fdlibm.lisp
=====================================
@@ -345,7 +345,9 @@
   ;; acosh(1.5) = log((sqrt(5)+3)/2, case 1 < x < 2
   (assert-eql 0.9624236501192069d0 (acosh 1.5d0))
   ;; acosh(4) = log(sqrt(15)+4), case 2 < x < 2^28
-  (assert-eql 2.0634370688955608d0 (acosh 4d0))
+  (assert-eql #-core-math 2.0634370688955608d0
+	      #+core-math 2.0634370688955603d0
+	      (acosh 4d0))
   ;; acosh(2^50), case 2^28 < x
   (assert-eql 35.35050620855721d0 (acosh (scale-float 1d0 50)))
   ;; No overflow for most positive



View it on GitLab: https://gitlab.common-lisp.net/cmucl/cmucl/-/compare/07fd3bac2529867a46d9dd6...

-- 
View it on GitLab: https://gitlab.common-lisp.net/cmucl/cmucl/-/compare/07fd3bac2529867a46d9dd6...
You're receiving this email because of your account on gitlab.common-lisp.net.

    

Raymond Toy (＠rtoy)

tags

participants (1)