[PATCH] [ASan] Fix use of -asan-instrument-assembly in tests

Jay Foad jay.foad at gmail.com
Mon Nov 10 01:45:18 PST 2014


Hi kcc, samsonov, eugenis,

The option -asan-instrument-assembly is declared in the X86 backend.
If I test on PowerPC configured with LLVM_TARGETS_TO_BUILD=All then the
option is tolerated but ignored.
If I test on PowerPC configured with LLVM_TARGETS_TO_BUILD=PowerPC then
the testsuite fails with:
[ 93%] Generating ASAN_INST_TEST_OBJECTS.gtest-all.cc.powerpc64-inline.o
clang (LLVM option parsing): Unknown command line argument '-asan-instrument-assembly'.  Try: 'clang (LLVM option parsing) -help'

Fix this inconsistency by only adding the option if that toolchain was
built with the X86 backend included.

http://reviews.llvm.org/D6190

Files:
  lib/asan/tests/CMakeLists.txt
-------------- next part --------------
A non-text attachment was scrubbed...
Name: D6190.15970.patch
Type: text/x-patch
Size: 551 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20141110/0ff69cc5/attachment.bin>


More information about the llvm-commits mailing list