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

Filipe Cabecinhas filcab+llvm.phabricator at gmail.com
Thu Feb 19 19:43:56 PST 2015


Wow, my face is red.
Thanks :-)

On Thu, Feb 19, 2015 at 7:14 PM, Alexey Samsonov <vonosmas at gmail.com> wrote:

> Please submit after fixing the error below:
>
>
> ================
> Comment at: test/CMakeLists.txt:13
> @@ +12,3 @@
> +  set_property(TARGET FileCheck PROPERTY IMPORTED_LOCATION
> ${LLVM_TOOLS_BINARY_DIR}/FileCheck)
> +  list(APPEND SANITIZER_COMMON_TEST_DEPS FileCheck)
> +endif()
> ----------------
> I believe this is still wrong =)
> Should be `SANITIZER_COMMON_LIT_TEST_DEPS`
>
> http://reviews.llvm.org/D7740
>
> EMAIL PREFERENCES
>   http://reviews.llvm.org/settings/panel/emailpreferences/
>
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20150219/daafd7d3/attachment.html>


More information about the llvm-commits mailing list