Skip to content

Commit d0c8406

Browse files
authored
Drop check for LLVM_LIBCXX in FindHalide_LLVM.cmake (#8617)
This was never tested in CI, and grepping the LLVM sources (18 - main) for this variable comes up empty. Mixing standard libraries in a single project isn't a well-supported scenario anyway, so users wishing to override the standard library should use a consistent toolchain file between LLVM and Halide to do so.
1 parent 58f36bf commit d0c8406

File tree

1 file changed

+0
-5
lines changed

1 file changed

+0
-5
lines changed

cmake/FindHalide_LLVM.cmake

Lines changed: 0 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -115,11 +115,6 @@ if (Halide_LLVM_FOUND)
115115
set_property(TARGET Halide_LLVM::Core PROPERTY INTERFACE_CXX_RTTI "${LLVM_ENABLE_RTTI}")
116116
set_property(TARGET Halide_LLVM::Core APPEND PROPERTY COMPATIBLE_INTERFACE_BOOL CXX_RTTI)
117117

118-
if (LLVM_LIBCXX GREATER -1)
119-
target_compile_options(Halide_LLVM::Core INTERFACE "$<$<LINK_LANGUAGE:CXX>:-stdlib=libc++>")
120-
target_link_options(Halide_LLVM::Core INTERFACE "$<$<LINK_LANGUAGE:CXX>:-stdlib=libc++>")
121-
endif ()
122-
123118
if (Halide_LLVM_SHARED_LIBS)
124119
target_link_libraries(Halide_LLVM::Core INTERFACE LLVM ${CMAKE_DL_LIBS})
125120
else ()

0 commit comments

Comments
 (0)