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

David CARLIER via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Jun 25 11:28:06 PDT 2018


devnexen added inline comments.


================
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:
> devnexen wrote:
> > 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.
> If you are not planing to do so soon, could you please comment TODO here, so we can remove it when we don't need it.
fair enough.


Repository:
  rCRT Compiler Runtime

https://reviews.llvm.org/D48528





More information about the llvm-commits mailing list