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

Linux/Arm64 build failed #9802

Open
odidev opened this issue May 14, 2024 · 5 comments
Open

Linux/Arm64 build failed #9802

odidev opened this issue May 14, 2024 · 5 comments
Labels
build triage Newly created issue that needs attention.

Comments

@odidev
Copy link

odidev commented May 14, 2024

Problem description

Hi @mbasmanova, when trying to build Velox from source on Linux/ARM64 using command CPU_TARGET="aarch64" make as mentioned in readme.md but the build fails. I am attaching the failure logs for the same. Could you please have a look into this issue?
I am trying to find out the version of Velox that supports Linux/ARM64.
Please let me know if any other information is required.

System information

Velox System Info v0.0.2
Commit: 85fd5c6
CMake Version: 3.22.1
System: Linux-6.5.0-1018-aws
Arch: aarch64
C++ Compiler: /usr/bin/c++
C++ Compiler Version: 11.4.0
C Compiler: /usr/bin/cc
C Compiler Version: 11.4.0
CMake Prefix Path: /usr/local;/usr;/;/usr;/usr/local;/usr/X11R6;/usr/pkg;/opt
\nThe results will be copied to your clipboard if xclip is installed.

CMake log

/home/ubuntu/velox/_build/release/_deps/xsimd-src/include/xsimd/arch/xsimd_scalar.hpp: In function ‘std::pair<double, double> xsimd::sincos(double)’: 
/home/ubuntu/velox/_build/release/_deps/xsimd-src/include/xsimd/arch/xsimd_scalar.hpp:988:57: note: parameter passing for argument of type ‘std::pair<double, double>’ when C++17 is enabled changed to match C++14 in GCC 10.1 
  988 |     inline std::pair<double, double> sincos(double val) noexcept 
      |                                                         ^~~~~~~~ 
[47/1338] Building CXX object velox/common/caching/tests/CMakeFiles/velox_cache_test.dir/AsyncDataCacheTest.cpp.o 
In file included from /home/ubuntu/velox/_build/release/_deps/xsimd-src/include/xsimd/xsimd.hpp:55, 
                 from /home/ubuntu/velox/./velox/common/base/SimdUtil.h:24, 
/home/ubuntu/velox/_build/release/_deps/xsimd-src/include/xsimd/arch/xsimd_scalar.hpp:981:54: note: parameter passing for argument of type ‘std::pair<float, float>’ when C++17 is enabled changed to match C++14 in GCC 10.1 
  981 |     inline std::pair<float, float> sincos(float val) noexcept 
      |                                                      ^~~~~~~~ 
/home/ubuntu/velox/_build/release/_deps/xsimd-src/include/xsimd/arch/xsimd_scalar.hpp: In function ‘std::pair<double, double> xsimd::sincos(double)’: 
/home/ubuntu/velox/_build/release/_deps/xsimd-src/include/xsimd/arch/xsimd_scalar.hpp:988:57: note: parameter passing for argument of type ‘std::pair<double, double>’ when C++17 is enabled changed to match C++14 in GCC 10.1 
  988 |     inline std::pair<double, double> sincos(double val) noexcept 
      |                                                         ^~~~~~~~ 
ninja: build stopped: subcommand failed. 
make[1]: *** [Makefile:103: build] Error 1 
make[1]: Leaving directory '/home/ubuntu/velox' 
make: *** [Makefile:110: release] Error 2
@odidev odidev added build triage Newly created issue that needs attention. labels May 14, 2024
@mbasmanova
Copy link
Contributor

CC: @kgpai @assignUser @majetideepak

@assignUser
Copy link
Collaborator

This is just a note and not what made the compilation fail. Could you post the rest of the log?

@odidev
Copy link
Author

odidev commented May 15, 2024

Here's the rest of the log: CMake Log.txt

@majetideepak
Copy link
Collaborator

In member function ‘void facebook::velox::SelectivityVector::clearAll()’,
    inlined from ‘std::enable_if_t<is_same_v<U, facebook::velox::StringView>, void> facebook::velox::SimpleVector<T>::invalidateIsAscii() [with U = facebook::velox::StringView; T = facebook::velox::StringView]’ at /home/ubuntu/velox/./velox/vector/SimpleVector.h:308:55,
    inlined from ‘void facebook::velox::SimpleVector<T>::resetDataDependentFlags(const facebook::velox::SelectivityVector*) [with T = facebook::velox::StringView]’ at /home/ubuntu/velox/./velox/vector/SimpleVector.h:427:26:
/home/ubuntu/velox/./velox/vector/SelectivityVector.h:159:18: error: writing 1 byte into a region of size 0 [-Werror=stringop-overflow=]

This is the error. While we fix the warning, can you try building by
TREAT_WARNINGS_AS_ERRORS=0 CPU_TARGET="aarch64" make

@majetideepak
Copy link
Collaborator

Here is the PR working on a fix for this issue #9578

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
build triage Newly created issue that needs attention.
Projects
None yet
Development

No branches or pull requests

4 participants