From 0ed6409618da9d7fa4f4250ab8bb6fa570988c09 Mon Sep 17 00:00:00 2001 From: Orgad Shaneh Date: Wed, 14 Nov 2018 23:52:34 +0200 Subject: Clang: Remove duplicate CXX flags Most of the flags appear twice on llvm-config-7 --cxxflags (at least on Debian). Change-Id: I9c5630d61472fa37bce998ec541a0ab561b83722 Reviewed-by: Nikolai Kosjar --- src/shared/clang/clang_installation.pri | 2 +- tests/unit/unittest/clang_dependency.pri | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/src/shared/clang/clang_installation.pri b/src/shared/clang/clang_installation.pri index 8f102279d4..afa21e23ce 100644 --- a/src/shared/clang/clang_installation.pri +++ b/src/shared/clang/clang_installation.pri @@ -216,7 +216,7 @@ isEmpty(LLVM_VERSION) { # Remove unwanted flags. It is a workaround for linking. # It is not intended for cross compiler linking. - LLVM_CXXFLAGS = $$system($$llvm_config --cxxflags, lines) + LLVM_CXXFLAGS *= $$system($$llvm_config --cxxflags, lines) LLVM_CXXFLAGS ~= s,-fno-exceptions, LLVM_CXXFLAGS ~= s,-std=c++11, LLVM_CXXFLAGS ~= s,-std=c++0x, diff --git a/tests/unit/unittest/clang_dependency.pri b/tests/unit/unittest/clang_dependency.pri index 744c9ff428..2c72feadff 100644 --- a/tests/unit/unittest/clang_dependency.pri +++ b/tests/unit/unittest/clang_dependency.pri @@ -22,7 +22,7 @@ include(../../../src/shared/clang/clang_defines.pri) !contains(QMAKE_DEFAULT_LIBDIRS, $$LLVM_LIBDIR): QMAKE_RPATHDIR += $$LLVM_LIBDIR LLVM_CXXFLAGS ~= s,-g\d?, - QMAKE_CXXFLAGS += $$LLVM_CXXFLAGS + QMAKE_CXXFLAGS *= $$LLVM_CXXFLAGS DEFINES += CLANG_COMPILER_PATH=\"R\\\"xxx($$LLVM_BINDIR/clang)xxx\\\"\" } -- cgit v1.2.1