Enable running Ubsan tests on FreeBSD

[kutuzov.viktor.84]

Summary

Will be landed after D6086, D6087 and D6088.

[emaste]

test/ubsan/lit.common.cfg

50 ↗

(On Diff #15710)

This comment is identical to the test that immediately follows so seems redundant.

Perhaps:

# Check that the host supports UndefinedBehaviorSanitizer
if config.host_os not in ...

[samsonov]

Sure, feel free to submit it once you fix the other stuff.

test/ubsan/lit.common.cfg

50 ↗

(On Diff #15710)

Or just delete this comment, the statement seems simple enough.

[kutuzov.viktor.84]

test/ubsan/lit.common.cfg

50 ↗

(On Diff #15710)

The comment itself looks to be useful as it refers to UndefinedBehaviorSanitizer tests explicitly while the code does not, but I agree that there's little sense in duplicating the list of supported platforms. I will follow the Ed's proposal, if there are no objections. Thanks.

Closed by commit rL221599 (authored by vkutuzov).

compiler-rt/trunk/test/ubsan/lit.common.cfg

 
 # Define %clang and %clangxx substitutions to use in test RUN lines.
 config.substitutions.append( ("%clang ", build_invocation(clang_ubsan_cflags)) )
 config.substitutions.append( ("%clangxx ", build_invocation(clang_ubsan_cxxflags)) )
 
 # Default test suffixes.
 config.suffixes = ['.c', '.cc', '.cpp']
 
-# UndefinedBehaviorSanitizer tests are currently supported on
-# Linux and Darwin only.
-if config.host_os not in ['Linux', 'Darwin']:
+# Check that the host supports UndefinedBehaviorSanitizer tests
+if config.host_os not in ['Linux', 'Darwin', 'FreeBSD']:
   config.unsupported = True
 
 # Allow tests to use REQUIRES=stable-runtime.  For use when you cannot use XFAIL
 # because the test hangs or fails on one configuration and not the other.
 if config.target_arch.startswith('arm') == False:
   config.available_features.add('stable-runtime')