[PATCH] add supporting of the sanitizer arguments into Clang on FreeBSD platform.

Alexey Samsonov samsonov at google.com
Mon Mar 17 05:48:37 PDT 2014


  LGTM


================
Comment at: lib/Driver/Tools.cpp:1780
@@ +1779,3 @@
+  StringRef OSLibName = Triple.getOS() == llvm::Triple::FreeBSD ?
+    Triple.getOSTypeName(Triple.getOS()) : TC.getOS();
+  llvm::sys::path::append(Res, "lib", OSLibName);
----------------
Yes, please use explicit "freebsd" for now (and leave a comment why you're doing this).

================
Comment at: lib/Driver/Tools.cpp:1843
@@ -1840,2 +1842,3 @@
   CmdArgs.push_back("-lm");
 
+  // There's no libdl on FreeBSD.
----------------
Remove newline here.


http://llvm-reviews.chandlerc.com/D2644



More information about the cfe-commits mailing list