Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Build fails on linux-32 #1737

Open
gsakkis opened this issue Jul 22, 2020 · 5 comments
Open

Build fails on linux-32 #1737

gsakkis opened this issue Jul 22, 2020 · 5 comments

Comments

@gsakkis
Copy link
Contributor

gsakkis commented Jul 22, 2020

I got the following error when trying to build on a linux-32 host:

/tmp/TileDB/external/src/blosc/shuffle.cc:280:4: error: #warning Hardware-acceleration detection not implemented for the target architecture. Only the generic shuffle/unshuffle routines will be available. [-Werror=cpp]
   #warning Hardware-acceleration detection not implemented for the target architecture. Only the generic shuffle/unshuffle routines will be available.
    ^
/tmp/TileDB/external/src/blosc/shuffle.cc: In function ‘blosc::shuffle_implementation_t blosc::get_shuffle_implementation()’:
/tmp/TileDB/external/src/blosc/shuffle.cc:290:22: error: unused variable ‘cpu_features’ [-Werror=unused-variable]
   blosc_cpu_features cpu_features = blosc_get_cpu_features();
                      ^
cc1plus: all warnings being treated as errors
@Shelnutt2
Copy link
Member

@gsakkis you can disable warnings as error to avoid this as a workaround. Please try adding -DTILEDB_WERROR to your cmake command or --disable-werror if you use the bootstrap script.

We will look at fixing the warning so you will not need to disable the werror for the future.

As a general note, we have not verbosely tested 32bit architectures, please let us know if you run into any further problems.

@gsakkis
Copy link
Contributor Author

gsakkis commented Jul 22, 2020

@Shelnutt2 thanks for the prompt response. I managed to built tiledb and tiledb-py with -DTILEDB_WERROR=ON (had to also pass -DTILEDB_S3=OFF).

Btw I got quite a few more warnings for tiledb-py and many tests fail with tiledb.libtiledb.TileDBError: [TileDB::Buffer] Error: Read failed; Trying to read beyond buffer size.

@Shelnutt2
Copy link
Member

@gsakkis thanks for reporting this. @joe-maley was able to track down the error and it should be fixed by #1739 . We will include this in the next 2.0.7 release.

@gsakkis
Copy link
Contributor Author

gsakkis commented Jul 24, 2020

Great, thanks a lot!

@jeroen
Copy link
Contributor

jeroen commented Aug 24, 2020

Try setting these before building:

export CFLAGS="-mfpmath=sse -msse2"
export CXXFLAGS="-mfpmath=sse -msse2"

See also: #1779

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

No branches or pull requests

3 participants