diff options
| author | Chris Bieneman <beanz@apple.com> | 2016-10-18 23:54:28 +0000 |
|---|---|---|
| committer | Chris Bieneman <beanz@apple.com> | 2016-10-18 23:54:28 +0000 |
| commit | f50761f128950bfedbba7f9ab56d28e2b33f29dd (patch) | |
| tree | fc7140ed191c48f325d110646f89554b15155941 /llvm/lib/Transforms/Utils/ASanStackFrameLayout.cpp | |
| parent | e80f007dfb6b4f04fb00b28559ce21a2852b89b5 (diff) | |
| download | llvm-f50761f128950bfedbba7f9ab56d28e2b33f29dd.zip llvm-f50761f128950bfedbba7f9ab56d28e2b33f29dd.tar.gz llvm-f50761f128950bfedbba7f9ab56d28e2b33f29dd.tar.bz2 | |
[CMake] Don't include LLDB_TEST_COMPILER in cached variable
Summary:
CMake has no builtin mechanism for cache invalidation. As a general convention you want to not expand user-specified variables in other cached variables because they will not get updated when the user changes their specified value.
This patch moves the "-C" option for dotest.py into the LLDB_TEST_COMMON_ARGS and out of the CMake cache. In order to prevent issues with out-of-date cache files on builders I've added code to scrub "-C ${LLDB_TEST_COMPILER}" out of the CMake caches, by Force writing the variable. This code can be removed in a few days once the change has trickled through CI systems.
Reviewers: tfiala, labath, zturner
Subscribers: lldb-commits, mgorny
Differential Revision: https://reviews.llvm.org/D25751
llvm-svn: 284551
Diffstat (limited to 'llvm/lib/Transforms/Utils/ASanStackFrameLayout.cpp')
0 files changed, 0 insertions, 0 deletions