[PATCH] D139782: [scudo][standalone] Use CheckAtomic to decide to link to libatomic

Fri Sep 22 18:20:40 PDT 2023

vitalybuka added a comment.

  cmake -B ../out/t -GNinja -S llvm '-DLLVM_ENABLE_PROJECTS=clang;lld' '-DLLVM_ENABLE_RUNTIMES=libcxx;libcxxabi;compiler-rt'   -DLLVM_CCACHE_BUILD=ON -DLLVM_ENABLE_PER_TARGET_RUNTIME_DIR=OFF  -DCMAKE_C_COMPILER=clang -DCMAKE_CXX_COMPILER=clang++  -DCMAKE_BUILD_TYPE=Release 

  ninja -C ../out/t 

  -- Performing Test HAVE_CXX_ATOMICS_WITH_LIB
  -- Performing Test HAVE_CXX_ATOMICS_WITH_LIB - Failed
  CMake Error at llvm-project/llvm/cmake/modules/CheckAtomic.cmake:57 (message):
    Host compiler must support std::atomic!
  Call Stack (most recent call first):
    llvm-project/compiler-rt/lib/scudo/standalone/CMakeLists.txt:3 (include)

  cat out/t/runtimes/runtimes-bins/CMakeFiles/CMakeError.log

  out/t/./bin/clang++ --target=x86_64-unknown-linux-gnu -DHAVE_CXX_ATOMICS_WITH_LIB  -fPIC -fno-semantic-interposition -fvisibility-inlines-hidden -Werror=date-time -Werror=unguarded-availability-new -Wall -Wextra -Wno-unused-parameter -Wwrite-strings -Wcast-qual -Wmissing-field-initializers -Wimplicit-fallthrough -Wcovered-switch-default -Wno-noexcept-type -Wnon-virtual-dtor -Wdelete-non-virtual-dtor -Wsuggest-override -Wstring-conversion -Wmisleading-indentation -Wctad-maybe-unsupported -fdiagnostics-color -ffunction-sections -fdata-sections -Wall -Wno-unused-parameter  --start-no-unused-arguments --unwindlib=none --end-no-unused-arguments -nostdlib++ -nostdinc++ -nodefaultlibs -std=c++11 -std=c++17 -MD -MT CMakeFiles/cmTC_6228f.dir/src.cxx.o -MF CMakeFiles/cmTC_6228f.dir/src.cxx.o.d -o CMakeFiles/cmTC_6228f.dir/src.cxx.o -c out/t/runtimes/runtimes-bins/CMakeFiles/CMakeScratch/TryCompile-1c0GZ3/src.cxx
  out/t/runtimes/runtimes-bins/CMakeFiles/CMakeScratch/TryCompile-1c0GZ3/src.cxx:2:10: fatal error: 'atomic' file not found
      2 | #include <atomic>
        |          ^~~~~~~~
  1 error generated.

Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D139782/new/

https://reviews.llvm.org/D139782