GNU bug report logs - #77404
[PATCH 0/1] PETSc compiler optimization: Pass from -O1 to -O3

Previous Next

Package: guix-patches;

Reported by: Luca Cirrottola <luca.cirrottola <at> inria.fr>

Date: Mon, 31 Mar 2025 10:43:04 UTC

Severity: normal

Tags: patch

Done: Andreas Enge <andreas <at> enge.fr>

To reply to this bug, email your comments to 77404 AT debbugs.gnu.org.
There is no need to reopen the bug first.

Toggle the display of automated, internal messages from the tracker.

View this report as an mbox folder, status mbox, maintainer mbox


Report forwarded to guix-patches <at> gnu.org:
bug#77404; Package guix-patches. (Mon, 31 Mar 2025 10:43:05 GMT) Full text and rfc822 format available.

Acknowledgement sent to Luca Cirrottola <luca.cirrottola <at> inria.fr>:
New bug report received and forwarded. Copy sent to guix-patches <at> gnu.org. (Mon, 31 Mar 2025 10:43:05 GMT) Full text and rfc822 format available.

Message #5 received at submit <at> debbugs.gnu.org (full text, mbox):

From: Luca Cirrottola <luca.cirrottola <at> inria.fr>
To: guix-patches <at> gnu.org
Cc: Luca Cirrottola <luca.cirrottola <at> inria.fr>
Subject: [PATCH 0/1] PETSc compiler optimization: Pass from -O1 to -O3
Date: Mon, 31 Mar 2025 12:42:17 +0200
Hello Guix,

I have recently noticed that the "petsc" package raises the following warnings during the configuration phase, showing that the compiler only does O1 optimization for now:

=============================================================================================
                         Configuring PETSc to compile on your system
=============================================================================================
=============================================================================================
                                     ***** WARNING *****
  Using default C optimization flags "-g -O". You might consider manually setting optimal
  optimization flags for your system with COPTFLAGS="optimization flags" see
  config/examples/arch-*-opt.py for examples
=============================================================================================
=============================================================================================
                                     ***** WARNING *****
  Using default Cxx optimization flags "-g". You might consider manually setting optimal
  optimization flags for your system with CXXOPTFLAGS="optimization flags" see
  config/examples/arch-*-opt.py for examples
=============================================================================================
=============================================================================================
                                     ***** WARNING *****
  Using default FC optimization flags "-g -O". You might consider manually setting optimal
  optimization flags for your system with FOPTFLAGS="optimization flags" see
  config/examples/arch-*-opt.py for examples
=============================================================================================

So here is a patch that follows the hints and passes the "-g -O3" flags to the COPTFLAGS, CXXOPTFLAGS and FOPTFLAGS variables.
The package builds successfully on my machine. Some tests fail for both the packages "petsc" and "petsc-openmpi", but these were failing already with the "-O1" flag (so maybe this could be the topic of another issue).
I am looking forward for your review.

Cheers,
Luca

Luca Cirrottola (1):
  Activate higher compiler optimization for PETSc.

 gnu/packages/maths.scm | 5 ++++-
 1 file changed, 4 insertions(+), 1 deletion(-)


base-commit: 4b97ae180fc13f7e4818cf48519f8a77c22c7797
-- 
2.34.1





Information forwarded to guix-patches <at> gnu.org:
bug#77404; Package guix-patches. (Mon, 31 Mar 2025 10:45:02 GMT) Full text and rfc822 format available.

Message #8 received at 77404 <at> debbugs.gnu.org (full text, mbox):

From: Luca Cirrottola <luca.cirrottola <at> inria.fr>
To: 77404 <at> debbugs.gnu.org
Cc: Luca Cirrottola <luca.cirrottola <at> inria.fr>
Subject: [PATCH 1/1] Activate higher compiler optimization for PETSc.
Date: Mon, 31 Mar 2025 12:44:35 +0200
---
 gnu/packages/maths.scm | 5 ++++-
 1 file changed, 4 insertions(+), 1 deletion(-)

diff --git a/gnu/packages/maths.scm b/gnu/packages/maths.scm
index 74c486b38a..fd0c940507 100644
--- a/gnu/packages/maths.scm
+++ b/gnu/packages/maths.scm
@@ -3658,7 +3658,10 @@ (define-public petsc
       #:test-target "test"
       #:parallel-build? #f             ; build is parallel by default
       #:configure-flags
-      #~(list "--with-mpi=0"
+      #~(list "COPTFLAGS=-g -O3"
+              "CXXOPTFLAGS=-g -O3"
+              "FOPTFLAGS=-g -O3"
+              "--with-mpi=0"
               "--with-openmp=1"
               "--with-openblas=1"
               (string-append "--with-openblas-dir="
-- 
2.34.1





Reply sent to Andreas Enge <andreas <at> enge.fr>:
You have taken responsibility. (Mon, 31 Mar 2025 14:08:04 GMT) Full text and rfc822 format available.

Notification sent to Luca Cirrottola <luca.cirrottola <at> inria.fr>:
bug acknowledged by developer. (Mon, 31 Mar 2025 14:08:05 GMT) Full text and rfc822 format available.

Message #13 received at 77404-done <at> debbugs.gnu.org (full text, mbox):

From: Andreas Enge <andreas <at> enge.fr>
To: 77404-done <at> debbugs.gnu.org
Cc: Luca Cirrottola <luca.cirrottola <at> inria.fr>
Subject: Close
Date: Mon, 31 Mar 2025 16:06:42 +0200
Hello,

thanks for the patch! I have pushed it after adapting the commit message
to our coding standards.

Andreas





This bug report was last modified 4 days ago.

Previous Next


GNU bug tracking system
Copyright (C) 1999 Darren O. Benham, 1997,2003 nCipher Corporation Ltd, 1994-97 Ian Jackson.