[PATCH] Enable running Ubsan tests on FreeBSD

Alexey Samsonov vonosmas at gmail.com
Mon Nov 3 11:24:02 PST 2014


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

================
Comment at: test/ubsan/lit.common.cfg:50
@@ -49,3 +49,3 @@
 # UndefinedBehaviorSanitizer tests are currently supported on
-# Linux and Darwin only.
-if config.host_os not in ['Linux', 'Darwin']:
+# Linux, Darwin and FreeBSD only.
+if config.host_os not in ['Linux', 'Darwin', 'FreeBSD']:
----------------
emaste wrote:
> 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 ...
> ```
Or just delete this comment, the statement seems simple enough.

http://reviews.llvm.org/D6089






More information about the llvm-commits mailing list