[sanitizer_common tests] Make Darwin a Posix system and bring the stable-runtime definition from ASan tests.

[filcab]

Summary

This is an initial setup in order to move some additional tests from Linux onto Posix.

[filcab]

I will probably add freebsd to the unsupported line in
decorate_proc_maps.cc since procfs isn't mounted by default. Or we can
just move that test to /Linux…

Thank you,

Filipe

[samsonov]

test/sanitizer_common/TestCases/Posix/decorate_proc_maps.cc

3

Right, let's move this to Linux/ directory instead.

test/sanitizer_common/lit.common.cfg

13

Why add this only for asan?

[filcab]

Address Samsonov's comments.

[filcab]

Updated with MSan's definition of stable-runtime

[samsonov]

test/sanitizer_common/lit.common.cfg

13

I mean, can you factor out this definition outside of "config.tool_name" conditions? It won't do harm for tsan/msan, as we don't build them for arm and armhf anyway, and I'm not sure we have a decent support for LSan as well.

[filcab]

Fair enough. I'll just factor out the definition and put the ARM
variants in the !stable-runtime camp.

Will upload in a bit.

Thank you,

Filipe

[filcab]

Factored out the definition of stable-runtime

[samsonov]

LGTM with a nit. Thanks!

test/sanitizer_common/lit.common.cfg

29

if config.target_arch not in ['arm', 'armhf', 'aarch64']:
  config.available_features.add('stable-runtime')

?

test/sanitizer_common/TestCases/Posix/decorate_proc_maps.cc

 // RUN: %clangxx -g %s -o %t
 // RUN: %env_tool_opts=decorate_proc_maps=1 %run %t 2>&1 | FileCheck %s --check-prefix=CHECK-%tool_name
+// UNSUPPORTED: darwin
 // REQUIRES: stable-runtime
 #include <errno.h>
 #include <fcntl.h>
 #include <pthread.h>
 #include <stdio.h>
 #include <sys/stat.h>
 #include <sys/types.h>
 #include <unistd.h>

test/sanitizer_common/TestCases/Posix/lit.local.cfg

 def getRoot(config):
   if not config.parent:
     return config
   return getRoot(config.parent)
 
 root = getRoot(config)
 
-if root.host_os in ['Windows', 'Darwin']:
+if root.host_os in ['Windows']:
   config.unsupported = True

test/sanitizer_common/lit.common.cfg

 # -*- Python -*-
 
 # Setup source root.
 config.test_source_root = os.path.join(os.path.dirname(__file__), "TestCases")
 
 config.name = "SanitizerCommon-" + config.name_suffix
 
 default_tool_options = []
 if config.tool_name == "asan":
   tool_cflags = ["-fsanitize=address"]
   tool_options = "ASAN_OPTIONS"
+  # Get the same definition from asan's lit.cfg
+  if config.target_arch != 'arm' and config.target_arch != 'armhf' and config.target_arch != 'aarch64':
+    config.available_features.add('stable-runtime')
 elif config.tool_name == "tsan":
   tool_cflags = ["-fsanitize=thread"]
   tool_options = "TSAN_OPTIONS"
 elif config.tool_name == "msan":
   tool_cflags = ["-fsanitize=memory"]
   tool_options = "MSAN_OPTIONS"
 elif config.tool_name == "lsan":
   tool_cflags = ["-fsanitize=leak"]
   tool_options = "LSAN_OPTIONS"
 else:
   lit_config.fatal("Unknown tool for sanitizer_common tests: %r" % config.tool_name)
 
 config.available_features.add(config.tool_name)
 
 if config.host_os == 'Darwin':
   # On Darwin, we default to `abort_on_error=1`, which would make tests run
   # much slower. Let's override this and run lit tests with 'abort_on_error=0'.
   default_tool_options += ['abort_on_error=0']
 default_tool_options_str = ':'.join(default_tool_options)
 if default_tool_options_str:
   config.environment[tool_options] = default_tool_options_str
   default_tool_options_str += ':'