diff mbox series

[v2] meson: Link against libc++ with compiling with clang

Message ID 20190619154056.32215-1-laurent.pinchart@ideasonboard.com (mailing list archive)
State Not Applicable, archived
Delegated to: Kieran Bingham
Headers show
Series [v2] meson: Link against libc++ with compiling with clang | expand

Commit Message

Laurent Pinchart June 19, 2019, 3:40 p.m. UTC
libc++ is used by Chrome OS, and likely as well by other environments
based on clang. Using it by default if available when compiling with
clang will help extending the compile-testing coverage. The drawback is
that issues specific to clang with libstdc++ will not be caught as
easily, but based on the experience with clang so far, code compiling
correctly with gcc/libstdc++ has failed with clang due to either
clang-specific or libc++-specific behaviour, never due only to the
combination of clang with libstdc++.

Signed-off-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com>
---
Changes since v1:

- Make the libc++ dependency optional
---
 meson.build | 9 +++++++++
 1 file changed, 9 insertions(+)
diff mbox series

Patch

diff --git a/meson.build b/meson.build
index 4d3e99d3e58f..a3b0bc820072 100644
--- a/meson.build
+++ b/meson.build
@@ -28,8 +28,17 @@  common_arguments = [
 c_arguments = common_arguments
 cpp_arguments = common_arguments
 
+# Use libc++ by default if available instead of libstdc++ when compiling with
+# clang.
+if cc.get_id() == 'clang' and cc.find_library('libc++', required: false).found()
+    cpp_arguments += [
+        '-stdlib=libc++',
+    ]
+endif
+
 add_project_arguments(c_arguments, language : 'c')
 add_project_arguments(cpp_arguments, language : 'cpp')
+add_project_link_arguments(cpp_arguments, language : 'cpp')
 
 libcamera_includes = include_directories('include')