[test-suite] r266769 - TEST_SUITE_DIAGNOSE_XXX should be the cached vars
Matthias Braun via llvm-commits
llvm-commits at lists.llvm.org
Tue Apr 19 10:28:08 PDT 2016
Author: matze
Date: Tue Apr 19 12:28:08 2016
New Revision: 266769
URL: http://llvm.org/viewvc/llvm-project?rev=266769&view=rev
Log:
TEST_SUITE_DIAGNOSE_XXX should be the cached vars
CMAKE_C_FLAGS etc. already are cached and need no alternate description.
Modified:
test-suite/trunk/CMakeLists.txt
Modified: test-suite/trunk/CMakeLists.txt
URL: http://llvm.org/viewvc/llvm-project/test-suite/trunk/CMakeLists.txt?rev=266769&r1=266768&r2=266769&view=diff
==============================================================================
--- test-suite/trunk/CMakeLists.txt (original)
+++ test-suite/trunk/CMakeLists.txt Tue Apr 19 12:28:08 2016
@@ -100,13 +100,17 @@ set(TEST_SUITE_PROFILE_USE "FALSE" CACHE
"Add apropriate -fprofile-instr-use to CFLAGS/CXXFLAGS for each benchmark")
# When running the test-suite in diagnosis mode, use these flags passed by
-# LNT to gather data, for examples -ftime-report, or -mllvm -stats.
-set(CMAKE_C_FLAGS "${CMAKE_C_FLAGS} ${TEST_SUITE_DIAGNOSE_FLAGS}" CACHE STRING
- "Extra CFLAGS for external tools like LNT.")
-set(CMAKE_CXX_FLAGS "${CMAKE_CXX_FLAGS} ${TEST_SUITE_DIAGNOSE_FLAGS}" CACHE STRING
- "Extra CXXFLAGS for external tools like LNT.")
-set(CMAKE_LD_FLAGS "${CMAKE_LD_FLAGS} ${TEST_SUITE_DIAGNOSE_FLAGS}" CACHE STRING
- "Extra LDFLAGS for external tools like LNT.")
+# LNT to gather data, for examples -ftime-report, or -mllvm -stats. This way
+# the user specified CMAKE_C_FLAGS etc. need not be changed.
+set(TEST_SUITE_DIAGNOSE_FLAGS CACHE STRING
+ "Extra flags appended to CMAKE_C_FLAGS + CMAKE_CXX_FLAGS")
+set(TEST_SUITE_DIAGNOSE_LINKER_FLAGS CACHE STRING
+ "Extra flags appended to CMAKE_EXE_LINKER_FLAGS")
+mark_as_advanced(TEST_SUITE_DIAGNOSE_FLAGS TEST_SUITE_DIAGNOSE_LINKER_FLAGS)
+set(CMAKE_C_FLAGS "${CMAKE_C_FLAGS} ${TEST_SUITE_DIAGNOSE_FLAGS}")
+set(CMAKE_CXX_FLAGS "${CMAKE_CXX_FLAGS} ${TEST_SUITE_DIAGNOSE_FLAGS}")
+set(CMAKE_EXE_LINKER_FLAGS
+ "${CMAKE_EXE_LINKER_FLAGS} ${TEST_SUITE_DIAGNOSE_LINKER_FLAGS}")
include(MakefileFunctions)
include(SingleMultiSource)
More information about the llvm-commits
mailing list