[PATCH] D48528: [UBsan] Enable ubsan minimal unit tests on OpenBSD

[David CARLIER via Phabricator via llvm-commits]

devnexen added inline comments.


================
Comment at: cmake/Modules/SanitizerUtils.cmake:84
 # Add target to check code style for sanitizer runtimes.
-if(CMAKE_HOST_UNIX)
+if(CMAKE_HOST_UNIX AND NOT OS_NAME MATCHES "OpenBSD")
   add_custom_target(SanitizerLintCheck
----------------
vitalybuka wrote:
> Why? It just needs a python.
The scripts can t works not because of Python.


================
Comment at: test/ubsan_minimal/lit.common.cfg:26
+if config.host_os == 'OpenBSD':
+  clang_ubsan_cflags.append("-fuse-ld=lld")
+  clang_ubsan_cxxflags.append("-fuse-ld=lld")
----------------
vitalybuka wrote:
> krytarowski wrote:
> > vitalybuka wrote:
> > > Why is OpenBSD special here?
> > I guess that it uses OpenBSD's gplv2 gnu ld that is incompatible with the modern world.
> Should it use lld by default, from driver?
I thought about too indeed.


Repository:
  rCRT Compiler Runtime

https://reviews.llvm.org/D48528