[PATCH] D55916: [clang] Replace getOS() == llvm::Triple::*BSD with isOS*BSD() [NFCI]

Ulrich Weigand via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Thu Dec 20 05:39:00 PST 2018


uweigand added a comment.

This causes test case failures due to no longer linking with -lrt on Linux.



================
Comment at: lib/Driver/ToolChains/CommonArgs.cpp:606
     CmdArgs.push_back("-lpthread");
-    if (TC.getTriple().getOS() != llvm::Triple::OpenBSD)
+    if (TC.getTriple().isOSOpenBSD())
       CmdArgs.push_back("-lrt");
----------------
Looks like this is missing a ! here.


Repository:
  rC Clang

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D55916/new/

https://reviews.llvm.org/D55916





More information about the cfe-commits mailing list