GNU bug report logs - #78334
[PATCH science-team 0/1] Update flint.

Previous Next

To reply to this bug, email your comments to 78334 AT debbugs.gnu.org.

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

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

From: Greg Hogan <code <at> greghogan.com>
To: guix-patches <at> gnu.org
Cc: Greg Hogan <code <at> greghogan.com>
Subject: [PATCH science-team 0/1] Update flint.
Date: Fri,  9 May 2025 15:39:18 +0000

Our current version, flint 3.1.2, fails for me with:

  CC  fft_small/mpn_mul.c
In file included from src/fft_small/mpn_mul.c:17:
src/fft_small/mpn_mul.c: In function ‘_mpn_from_ffts_8’:
./src/crt_helpers.h:120:3: error: ‘asm’ operand has impossible constraints
  120 |   __asm__ ("addq %23,%q7\nadcq %21,%q6\nadcq %19,%q5\n\tadcq %17,%q4\n\tadcq %15,%q3\n\tadcq %13,%q2\n\tadcq %11,%q1\n\tadcq %9,%q0"    \
      |   ^~~~~~~
./src/crt_helpers.h:378:5: note: in expansion of macro ‘add_ssssssssaaaaaaaaaaaaaaaa’
  378 |     add_ssssssssaaaaaaaaaaaaaaaa(z[7],z[6],z[5],z[4],z[3],z[2],z[1],z[0],
      |     ^~~~~~~~~~~~~~~~~~~~~~~~~~~~
./src/crt_helpers.h:120:3: error: ‘asm’ operand has impossible constraints
  120 |   __asm__ ("addq %23,%q7\nadcq %21,%q6\nadcq %19,%q5\n\tadcq %17,%q4\n\tadcq %15,%q3\n\tadcq %13,%q2\n\tadcq %11,%q1\n\tadcq %9,%q0"    \
      |   ^~~~~~~
./src/crt_helpers.h:378:5: note: in expansion of macro ‘add_ssssssssaaaaaaaaaaaaaaaa’
  378 |     add_ssssssssaaaaaaaaaaaaaaaa(z[7],z[6],z[5],z[4],z[3],z[2],z[1],z[0],
      |     ^~~~~~~~~~~~~~~~~~~~~~~~~~~~
make: *** [Makefile:616: build/fft_small/mpn_mul.lo] Error 1
error: in phase 'build': uncaught exception:
%exception #<&invoke-error program: "make" arguments: ("-j" "1") exit-status: 2 term-signal: #f stop-signal: #f>
phase `build' failed after 260.7 seconds
command "make" "-j" "1" failed with status 2
build process 18 exited with status 256

Greg Hogan (1):
  gnu: flint: Update to 3.2.1.

 gnu/packages/algebra.scm | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)


base-commit: c3b6b57521404d223ae37d3dd91f7bb60ed0cebc
-- 
2.49.0

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

From: Greg Hogan <code <at> greghogan.com>
To: 78334 <at> debbugs.gnu.org
Cc: Greg Hogan <code <at> greghogan.com>
Subject: [PATCH science-team 1/1] gnu: flint: Update to 3.2.1.
Date: Fri,  9 May 2025 15:40:31 +0000

* gnu/packages/algebra.scm (flint): Update to 3.2.1.
[source]: Update uri path.

Change-Id: I51adce1405811adf0aec48f75ca10cb9ecb03296
---
 gnu/packages/algebra.scm | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/gnu/packages/algebra.scm b/gnu/packages/algebra.scm
index f928e7943c..ab2ec5e327 100644
--- a/gnu/packages/algebra.scm
+++ b/gnu/packages/algebra.scm
@@ -485,13 +485,13 @@ (define-public giac
 (define-public flint
   (package
    (name "flint")
-   (version "3.1.2")
+   (version "3.2.1")
    (source
     (origin
       (method url-fetch)
-      (uri (string-append "https://flintlib.org/flint-" version ".tar.gz"))
+      (uri (string-append "https://flintlib.org/download/flint-" version ".tar.gz"))
       (sha256
-       (base32 "0017bjncpx4kdx67qcnm3nahz3gyyi2w3ggkrx586r3llcqs9czx"))))
+       (base32 "0gyjbkhwrmx2vgb1gailnmmzacl4aikzgi70dzmpf8lpfxny8yya"))))
    (build-system gnu-build-system)
    (inputs
     (list ntl))
-- 
2.49.0

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

From: Andreas Enge <andreas <at> enge.fr>
To: Greg Hogan <code <at> greghogan.com>
Cc: Sharlatan Hellseher <sharlatanus <at> gmail.com>, 78334 <at> debbugs.gnu.org,
 Eric Bavier <bavier <at> posteo.net>
Subject: Re: [bug#78334] [PATCH science-team 0/1] Update flint.
Date: Fri, 9 May 2025 17:52:54 +0200

Hello,

Am Fri, May 09, 2025 at 03:39:18PM +0000 schrieb Greg Hogan:
> Our current version, flint 3.1.2, fails for me with:
>   CC  fft_small/mpn_mul.c
> In file included from src/fft_small/mpn_mul.c:17:
> src/fft_small/mpn_mul.c: In function ‘_mpn_from_ffts_8’:
> ./src/crt_helpers.h:120:3: error: ‘asm’ operand has impossible constraints
>   120 |   __asm__ ("addq %23,%q7\nadcq %21,%q6\nadcq %19,%q5\n\tadcq %17,%q4\n\tadcq %15,%q3\n\tadcq %13,%q2\n\tadcq %11,%q1\n\tadcq %9,%q0"    \

we can update flint, but this message may indicate a different problem,
which surprises me. Flint includes assembly code, but does not use fat
binaries such as gmp. So it is possible that assembly code chosen on the
build machine (avx or whatever) is not executable on the machine where
you run your own code linked to flint.

But we build the package with
   #:configure-flags '("--disable-assembly")
so the problem should not occur.

Maybe it is indeed a bug in the version we currently package.

Andreas

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

From: Greg Hogan <code <at> greghogan.com>
To: Andreas Enge <andreas <at> enge.fr>
Cc: Sharlatan Hellseher <sharlatanus <at> gmail.com>, 78334 <at> debbugs.gnu.org,
 Eric Bavier <bavier <at> posteo.net>
Subject: Re: [bug#78334] [PATCH science-team 0/1] Update flint.
Date: Fri, 9 May 2025 11:55:57 -0400

On Fri, May 9, 2025 at 11:53 AM Andreas Enge <andreas <at> enge.fr> wrote:
>
> Hello,
>
> Am Fri, May 09, 2025 at 03:39:18PM +0000 schrieb Greg Hogan:
> > Our current version, flint 3.1.2, fails for me with:
> >   CC  fft_small/mpn_mul.c
> > In file included from src/fft_small/mpn_mul.c:17:
> > src/fft_small/mpn_mul.c: In function ‘_mpn_from_ffts_8’:
> > ./src/crt_helpers.h:120:3: error: ‘asm’ operand has impossible constraints
> >   120 |   __asm__ ("addq %23,%q7\nadcq %21,%q6\nadcq %19,%q5\n\tadcq %17,%q4\n\tadcq %15,%q3\n\tadcq %13,%q2\n\tadcq %11,%q1\n\tadcq %9,%q0"    \
>
> we can update flint, but this message may indicate a different problem,
> which surprises me. Flint includes assembly code, but does not use fat
> binaries such as gmp. So it is possible that assembly code chosen on the
> build machine (avx or whatever) is not executable on the machine where
> you run your own code linked to flint.

I'm building flint on the same machine as I built the rest of Guix.
And this is the flint build itself, not dependent code.

> But we build the package with
>    #:configure-flags '("--disable-assembly")
> so the problem should not occur.
>
> Maybe it is indeed a bug in the version we currently package.

v3.2.2 builds fine for me.

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

From: Andreas Enge <andreas <at> enge.fr>
To: Greg Hogan <code <at> greghogan.com>
Cc: Sharlatan Hellseher <sharlatanus <at> gmail.com>, 78334 <at> debbugs.gnu.org,
 Eric Bavier <bavier <at> posteo.net>
Subject: Re: [bug#78334] [PATCH science-team 0/1] Update flint.
Date: Mon, 12 May 2025 17:27:16 +0200

Unfortunately this flint update breaks singular:
facMul.cc: In function ‘void kronSubFp(nmod_poly_struct*, const CanonicalForm&, int)’:
facMul.cc:1258:3: error: ‘flint_mpn_zero’ was not declared in this scope
 1258 |   flint_mpn_zero (result->coeffs, d*(degAy+1));
      |   ^~~~~~~~~~~~~~
facMul.cc:1267:5: error: ‘flint_mpn_copyi’ was not declared in this scope
 1267 |     flint_mpn_copyi (result->coeffs+k, buf->coeffs, nmod_poly_length(buf));
      |     ^~~~~~~~~~~~~~~

Which also means it breaks sage. This is very annoying, we cannot just
push it, but need to wait for the depending packages to adapt to whatever
change was made. Or ship several versions.

Bug report submitted:
   https://github.com/flintlib/flint/issues/2315

Andreas

Previous Next

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