[PATCH] D16797: Update clang support on recent Haiku

Richard Smith via cfe-commits cfe-commits at lists.llvm.org
Thu Mar 24 14:35:53 PDT 2016


rsmith requested changes to this revision.
rsmith added a comment.
This revision now requires changes to proceed.

Please rebase against trunk. I'm surprised that no tests need updating due to this. Can you add some, please?


================
Comment at: lib/Basic/Targets.cpp:387
@@ +386,3 @@
+    this->TLSSupported = false;
+    this->UserLabelPrefix = "";
+
----------------
This doesn't exist here any more.

================
Comment at: lib/Driver/ToolChains.cpp:3004-3016
@@ +3003,15 @@
+
+  switch (GetCXXStdlibType(DriverArgs)) {
+  case ToolChain::CST_Libstdcxx:
+    addSystemInclude(DriverArgs, CC1Args,
+                     getDriver().SysRoot + "/system/develop/headers/c++");
+    addSystemInclude(DriverArgs, CC1Args,
+                     getDriver().SysRoot + "/system/develop/headers/c++/backward");
+
+    StringRef Triple = getTriple().str();
+    addSystemInclude(DriverArgs, CC1Args,
+                     getDriver().SysRoot + "/system/develop/headers/c++/" +
+                     Triple);
+    break;
+  }
+}
----------------
This switch should also cover `CST_Libcxx` somehow.


Repository:
  rL LLVM

http://reviews.llvm.org/D16797





More information about the cfe-commits mailing list