[UBSan] Fix typo in CMake conditional that checked if the architecture
authorDan Liew <dan@su-root.co.uk>
Wed, 19 Sep 2018 15:27:00 +0000 (15:27 +0000)
committerDan Liew <dan@su-root.co.uk>
Wed, 19 Sep 2018 15:27:00 +0000 (15:27 +0000)
of a darwin platform was in the list of `UBSAN_SUPPORTED_ARCH`.

This is a follow up to r341306.

The typo meant that if an architecture was a prefix to another
architecture in the list (e.g. `armv7` is a prefix of `armv7k`) then
this would trigger a match which is not the intended behaviour.

rdar://problem/41126835

llvm-svn: 342553

compiler-rt/test/ubsan/CMakeLists.txt

index d7f3c65..351bf3b 100644 (file)
@@ -117,7 +117,7 @@ if(APPLE)
       string(REPLACE ";" " " UBSAN_TEST_TARGET_ARCH_FLAGS "${UBSAN_TEST_TARGET_ARCH_FLAGS_AS_LIST}")
       set(UBSAN_TEST_TARGET_CFLAGS
         "${UBSAN_TEST_TARGET_ARCH_FLAGS} -isysroot ${DARWIN_${platform}_SYSROOT}")
-      if (";${UBSAN_SUPPORTED_ARCH};" MATCHES ";${arch}")
+      if (";${UBSAN_SUPPORTED_ARCH};" MATCHES ";${arch};")
         add_ubsan_device_testsuite("Standalone" ubsan ${platform} ${arch})
       endif()