Skip to content

Conversation

@dfukalov
Copy link
Collaborator

@dfukalov dfukalov commented Feb 7, 2024

Disable libstdc++ assertions in the runtime library just like in https://reviews.llvm.org/D143168.

Disable libstdc++ assertions in the runtime library just like in https://reviews.llvm.org/D143168.
@dfukalov dfukalov added cmake Build system in general and CMake in particular openmp openmp:libomp OpenMP host runtime labels Feb 7, 2024
@llvmbot
Copy link
Member

llvmbot commented Feb 7, 2024

@llvm/pr-subscribers-openmp

Author: Daniil Fukalov (dfukalov)

Changes

Disable libstdc++ assertions in the runtime library just like in https://reviews.llvm.org/D143168.


Full diff: https://github.com/llvm/llvm-project/pull/81029.diff

1 Files Affected:

  • (modified) openmp/runtime/src/CMakeLists.txt (+4)
diff --git a/openmp/runtime/src/CMakeLists.txt b/openmp/runtime/src/CMakeLists.txt index b0ecf1213dabe5..ff129feb5b6fa0 100644 --- a/openmp/runtime/src/CMakeLists.txt +++ b/openmp/runtime/src/CMakeLists.txt @@ -152,6 +152,10 @@ if(UNIX) set(LIBOMP_DL_LIBS ${CMAKE_DL_LIBS}) endif() +# Disable libstdc++ assertions, even in an LLVM_ENABLE_ASSERTIONS build, to +# avoid an unwanted dependency on libstdc++.so. +add_definitions(-U_GLIBCXX_ASSERTIONS) + # Add the OpenMP library libomp_get_ldflags(LIBOMP_CONFIGURED_LDFLAGS) 
@github-actions
Copy link

github-actions bot commented Feb 7, 2024

⚠️ We detected that you are using a GitHub private e-mail address to contribute to the repo.
Please turn off Keep my email addresses private setting in your account.
See LLVM Discourse for more information.

@dfukalov dfukalov merged commit 94272a5 into llvm:main Feb 9, 2024
@dfukalov dfukalov deleted the dfukalov/libomp_build_fix branch February 9, 2024 16:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

cmake Build system in general and CMake in particular openmp:libomp OpenMP host runtime openmp

3 participants