[PATCH] Make FileCheck be a common dependency, not an ASan one.

Filipe Cabecinhas filcab+llvm.phabricator at gmail.com
Thu Feb 19 18:15:52 PST 2015


Submitted a new patch with the fixed variable name


================
Comment at: test/CMakeLists.txt:13
@@ +12,3 @@
+  set_property(TARGET FileCheck PROPERTY IMPORTED_LOCATION ${LLVM_TOOLS_BINARY_DIR}/FileCheck)
+  list(APPEND ASAN_TEST_DEPS FileCheck)
+endif()
----------------
samsonov wrote:
> Shouldn't you append it to SANITIZER_COMMON_LIT_TEST_DEPS instead?
Yes, of course.

http://reviews.llvm.org/D7740

EMAIL PREFERENCES
  http://reviews.llvm.org/settings/panel/emailpreferences/






More information about the llvm-commits mailing list