Subject: bug#74791: closed (Re: [PATCH] transformations: Add tuning wrapper for gfortran.)

From: [email-masked] (GNU bug Tracking System)
Date: Thu, 12 Dec 2024 10:11:02 +0000

Your bug report

#74791: [PATCH] transformations: Add tuning wrapper for gfortran.

which was filed against the guix-patches package, has been closed.

The explanation is attached below, along with your original report.
If you require more details, please reply to [email-masked].

-- 
74791: https://debbugs.gnu.org/cgi/bugreport.cgi?bug=74791
GNU Bug Tracking System
Contact [email-masked] with problems

Subject: Re: [PATCH] transformations: Add tuning wrapper for gfortran.
From: Ludovic Courtès <[email-masked]>
To: Romain GARBAGE <[email-masked]>
Cc: [email-masked]
Date: Thu, 12 Dec 2024 11:09:59 +0100
List-Id: <debbugs-submit.debbugs.gnu.org>

Hello comrade,

Romain GARBAGE <[email-masked]> skribis:

> guix/transformations.scm (tuning-compiler): Add gfortran to the list
> of wrapped compilers.
>
> Change-Id: Ic7eaa07acbb5ea013af0d515686e961106a6bbbb

I added the ‘tunable?’ property to ‘lapack’ and confirmed that this
works in a crude but reliable way:

--8<---------------cut here---------------start------------->8---
$ pgrep -fa fortran |grep -e -march
26008 /gnu/store/piai5ba5fx901hr7pwnz2b9vysrrjmlh-gfortran-11.4.0/bin/gfortran -Dblas_EXPORTS -O2 -g -DNDEBUG -fPIC -c /tmp/guix-build-lapack-3.9.0.drv-0/lapack-3.9.0/BLAS/SRC/zher2.f -o CMakeFiles/blas.dir/zher2.f.o -march=skylake
26016 /gnu/store/piai5ba5fx901hr7pwnz2b9vysrrjmlh-gfortran-11.4.0/bin/gfortran -Dblas_EXPORTS -O2 -g -DNDEBUG -fPIC -c /tmp/guix-build-lapack-3.9.0.drv-0/lapack-3.9.0/BLAS/SRC/zhpr2.f -o CMakeFiles/blas.dir/zhpr2.f.o -march=skylake
26019 /gnu/store/piai5ba5fx901hr7pwnz2b9vysrrjmlh-gfortran-11.4.0/bin/gfortran -Dblas_EXPORTS -O2 -g -DNDEBUG -fPIC -c /tmp/guix-build-lapack-3.9.0.drv-0/lapack-3.9.0/BLAS/SRC/zgemm.f -o CMakeFiles/blas.dir/zgemm.f.o -march=skylake
--8<---------------cut here---------------end--------------->8---

Thanks!

Ludo’.

Subject: [PATCH] transformations: Add tuning wrapper for gfortran.
From: Romain GARBAGE <[email-masked]>
To: [email-masked]
Cc: [email-masked], Romain GARBAGE <[email-masked]>
Date: Wed, 11 Dec 2024 17:06:17 +0100
List-Id: <debbugs-submit.debbugs.gnu.org>

guix/transformations.scm (tuning-compiler): Add gfortran to the list
of wrapped compilers.

Change-Id: Ic7eaa07acbb5ea013af0d515686e961106a6bbbb
---
 guix/transformations.scm | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/guix/transformations.scm b/guix/transformations.scm
index 3f7d20beaa..131b8564f8 100644
--- a/guix/transformations.scm
+++ b/guix/transformations.scm
@@ -531,7 +531,7 @@ (define bin (string-append #$output "/bin"))
                                      (symlink #$program
                                               (string-append bin "/" program)))
                                    '("cc" "gcc" "clang" "g++" "c++" "clang++"
-                                     "go" "rustc" "zig")))))))
+                                     "gfortran" "go" "rustc" "zig")))))))
 
 (define (build-system-with-tuning-compiler bs micro-architecture)
   "Return a variant of BS, a build system, that ensures that the compiler that

base-commit: c2e2b0376b82a503e064fa826d1b153d94bf0b06
-- 
2.46.0