#17635: Update Givaro, FFLAS-FFPACK and LinBox
-------------------------------------+-------------------------------------
       Reporter:  jpflori            |        Owner:
           Type:  enhancement        |       Status:  needs_work
       Priority:  major              |    Milestone:  sage-7.2
      Component:  packages:          |   Resolution:
  standard                           |    Merged in:
       Keywords:                     |    Reviewers:  François Bissey,
        Authors:  Clement Pernet,    |  Jeroen Demeyer
  Jeroen Demeyer                     |  Work issues:
Report Upstream:  N/A                |       Commit:
         Branch:                     |  d0d3671efb3d418d29c01bb2def611f52d6218f8
  u/jdemeyer/givaro_fflasffpack_linbox|     Stopgaps:
   Dependencies:                     |
-------------------------------------+-------------------------------------

Comment (by fbissey):

 OS X: lots of problems with gmp c++ headers in `givaro`'s test suite.
 `givaro` builds and installs fine just the tests fail to build.
 {{{
 g++ -DHAVE_CONFIG_H -I. -I..  -O0 -Wall -g -DNDEBUG -UGIVARO_DEBUG -UDEBUG
 -DSTD_RECINT_SIZE=8 -I/Users/buildslave-
 sage/slave/sage_git/build/local//include -I.. -I../src/kernel/system
 -I../src/kernel/memory -I../src/kernel/field -I../src/kernel/ring
 -I../src/kernel/integer -I../src/kernel -I../src/library/poly1
 -I../src/kernel/bstruct -I../src/library/tools -I../src/kernel/recint  -O0
 -Wall -g -DNDEBUG -UGIVARO_DEBUG -UDEBUG -DNOT_IN_PLACE -DRI_OP=addmul
 -DGMP_OP=mpz_addmul -DGMP_OPUI=mpz_addmul_ui -I/Users/buildslave-
 sage/slave/sage_git/build/local//include  -fPIC -I"/Users/buildslave-
 sage/slave/sage_git/build/local/include" -std=gnu++11 -c -o
 test_ruint_arith_addmul-test-ruint_arith.o `test -f 'test-ruint_arith.C'
 || echo './'`test-ruint_arith.C
 In file included from ../src/kernel/recint/rudisplay.h:50:0,
                  from ../src/kernel/recint/ruint.h:73,
                  from ../src/kernel/recint/recint.h:45,
                  from test-rmint_arith.C:16:
 ../src/kernel/recint/ruconvert.h: In instantiation of 'mpz_class&
 RecInt::ruint_to_mpz(mpz_class&, const RecInt::ruint<K>&) [with long
 unsigned int K = 8ul; mpz_class = __gmp_expr<__mpz_struct [1],
 __mpz_struct [1]>]':
 test-rmint_arith.C:39:29:   required from here
 ../src/kernel/recint/ruconvert.h:99:15: error: ambiguous overload for
 'operator^=' (operand types are 'mpz_class {aka __gmp_expr<__mpz_struct
 [1], __mpz_struct [1]>}' and 'RecInt::limb {aka long long unsigned int}')
              a ^= static_cast<uint64_t>(*it);
                ^
 ../src/kernel/recint/ruconvert.h:99:15: note: candidates are:
 In file included from test-rmint_arith.C:15:0:
 /Users/buildslave-sage/slave/sage_git/build/local/include/gmpxx.h:3283:1:
 note: mpz_class& __gmp_expr<__mpz_struct [1], __mpz_struct
 [1]>::operator^=(signed char)
  __GMPZ_DEFINE_COMPOUND_OPERATOR(operator^=, __gmp_binary_xor)
  ^
 /Users/buildslave-sage/slave/sage_git/build/local/include/gmpxx.h:3283:1:
 note: mpz_class& __gmp_expr<__mpz_struct [1], __mpz_struct
 [1]>::operator^=(unsigned char)
  __GMPZ_DEFINE_COMPOUND_OPERATOR(operator^=, __gmp_binary_xor)
  ^
 }}}
 and it goes on.

 linux 32bits, the cpu is modern and offers sse4.1, avx and avx2 but is 32
 bits.
 {{{
 libtool: compile:  g++ -DHAVE_CONFIG_H -I. -I../../.. -I../../.. -I
 ../../../fflas-ffpack/utils/ -I../../../fflas-ffpack/fflas/ -I
 ../../../fflas-ffpack/ffpack -I../../../fflas-ffpack/field
 -I/home/buildbot/slave/sage_git/build/local/include
 -I/home/buildbot/slave/sage_git/build/local//include
 -D__FFLASFFPACK_HAVE_CBLAS -msse4.1 -mfma -mavx2 -fabi-version=6 -O2 -Wall
 -DNDEBUG -UFFLASFFPACK_DEBUG -g -O2 -fPIC -std=gnu++11 -c fflas_L2_inst.C
 -fPIC -DPIC -o .libs/fflas_L2_inst.o
 In file included from ../../../fflas-
 ffpack/fflas/fflas_simd/simd256.inl:44:0,
                  from ../../../fflas-ffpack/fflas/fflas_simd.h:208,
                  from ../../../fflas-ffpack/fflas/fflas_freduce.h:33,
                  from ../../../fflas-ffpack/fflas/fflas.h:99,
                  from fflas_L1_inst.C:34:
 ../../../fflas-ffpack/fflas/fflas_simd/simd256_int64.inl: In static member
 function 'static Simd256_impl<true, true, true, 8>::vect_t
 Simd256_impl<true, true, true, 8>::mulhi(Simd256_impl<true, true, true,
 8>::vect_t, Simd256_impl<true, true, true, 8>::vect_t)':
 ../../../fflas-ffpack/fflas/fflas_simd/simd256_int64.inl:246:37: error:
 'int128_t' was not declared in this scope
          return set((int128_t(ca.t[0]) * cb.t[0]) >> 64,
 (int128_t(ca.t[1]) * cb.t[1]) >> 64,
                                      ^
 In file included from ../../../fflas-
 ffpack/fflas/fflas_simd/simd256.inl:44:0,
                  from ../../../fflas-ffpack/fflas/fflas_simd.h:208,
                  from ../../../fflas-ffpack/fflas/fflas_freduce.h:33,
                  from ../../../fflas-ffpack/fflas/fflas.h:99,
                  from fflas_L2_inst.C:34:
 ../../../fflas-ffpack/fflas/fflas_simd/simd256_int64.inl: In static member
 function 'static Simd256_impl<true, true, true, 8>::vect_t
 Simd256_impl<true, true, true, 8>::mulhi(Simd256_impl<true, true, true,
 8>::vect_t, Simd256_impl<true, true, true, 8>::vect_t)':
 ../../../fflas-ffpack/fflas/fflas_simd/simd256_int64.inl:246:37: error:
 'int128_t' was not declared in this scope
          return set((int128_t(ca.t[0]) * cb.t[0]) >> 64,
 (int128_t(ca.t[1]) * cb.t[1]) >> 64,
                                      ^
 }}}
 There may be a wrong assumptions about type there.

--
Ticket URL: <http://trac.sagemath.org/ticket/17635#comment:136>
Sage <http://www.sagemath.org>
Sage: Creating a Viable Open Source Alternative to Magma, Maple, Mathematica, 
and MATLAB

-- 
You received this message because you are subscribed to the Google Groups 
"sage-trac" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to [email protected].
To post to this group, send email to [email protected].
Visit this group at https://groups.google.com/group/sage-trac.
For more options, visit https://groups.google.com/d/optout.

Reply via email to