Skip to content
This repository has been archived by the owner on Sep 28, 2021. It is now read-only.

2021-rc.1 fails to build on i586 #651

Closed
junghans opened this issue Jan 18, 2021 · 3 comments
Closed

2021-rc.1 fails to build on i586 #651

junghans opened this issue Jan 18, 2021 · 3 comments
Assignees

Comments

@junghans
Copy link
Member

[   60s] [  2%] Building CXX object src/libxtp/CMakeFiles/votca_xtp.dir/aobasis.cc.o
[   61s] cd /home/abuild/rpmbuild/BUILD/xtp-2021-rc.1/build/src/libxtp && /usr/bin/c++ -DBOOST_ALL_NO_LIB -DBOOST_CHRONO_DYN_LINK -DBOOST_FILESYSTEM_DYN_LINK -DBOOST_PROGRAM_OPTIONS_DYN_LINK -DBOOST_SYSTEM_DYN_LINK -DBOOST_TIMER_DYN_LINK -Dvotca_xtp_EXPORTS -I/home/abuild/rpmbuild/BUILD/xtp-2021-rc.1/build/src/libxtp -I/home/abuild/rpmbuild/BUILD/xtp-2021-rc.1/src/libxtp -I/home/abuild/rpmbuild/BUILD/xtp-2021-rc.1/include -I/home/abuild/rpmbuild/BUILD/xtp-2021-rc.1/build/include -I/home/abuild/rpmbuild/BUILD/xtp-2021-rc.1/build/include/votca/xtp -isystem /usr/include/eigen3 -isystem /usr/include/libint2 -fomit-frame-pointer -O2 -Wall -D_FORTIFY_SOURCE=2 -fstack-protector-strong -funwind-tables -fasynchronous-unwind-tables -fstack-clash-protection -Werror=return-type -flto=auto -g -DNDEBUG -O2 -g -DNDEBUG -fPIC -fopenmp -std=c++14 -o CMakeFiles/votca_xtp.dir/aobasis.cc.o -c /home/abuild/rpmbuild/BUILD/xtp-2021-rc.1/src/libxtp/aobasis.cc
[   92s] In file included from /usr/include/libint2/engine.impl.h:33,
[   92s]                  from /usr/include/libint2/engine.h:988,
[   92s]                  from /usr/include/libint2/cxxapi.h:41,
[   92s]                  from /usr/include/libint2.hpp:24,
[   92s]                  from /home/abuild/rpmbuild/BUILD/xtp-2021-rc.1/src/libxtp/aobasis.cc:26:
[   92s] /usr/include/libint2/boys.h: In member function 'void libint2::TennoGmEval<Real>::init_table()':
[   92s] /usr/include/libint2/boys.h:1291:73: error: no matching function for call to 'max(unsigned int, long unsigned int)'
[   92s]  1291 |         int status = posix_memalign(&result, std::max(sizeof(Real), 32ul), (mmax_ - mmin_ + 1) * cheb_table_nintervals * ORDERp1 * ORDERp1 * sizeof(Real));
[   92s]       |                                                                         ^

Some we pick up a unsigned int instead of a long unsigned int.

Details here

Please fix on stable.

@JensWehner
Copy link
Member

i586 is 32bit is it? I think this is where we drop 32bit support, because this is deep inside libint.

@junghans
Copy link
Member Author

Seems reasonable!

A simple program:

#include <libint2.hpp>
int main(){return 0;}

fails with the same error.

@junghans
Copy link
Member Author

This is evaleev/libint#194.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants