[PATCH] [MIPS] Add support for direct-to-nacl in Clang

Derek Schuff dschuff at google.com
Thu Jun 25 11:17:14 PDT 2015


REPOSITORY
  rL LLVM

================
Comment at: lib/Driver/ToolChains.cpp:2391
@@ +2390,3 @@
+      file_paths.push_back(FilePath + "mipsel-nacl/usr/lib");
+      prog_paths.push_back(ProgPath + "bin");
+      file_paths.push_back(ToolPath + "mipsel-nacl");
----------------
should this be mipsel-nacl/bin?

================
Comment at: lib/Driver/ToolChains.cpp:2416
@@ +2415,3 @@
+    SmallString<128> P(D.Dir + "/../");
+    llvm::sys::path::append(P, "mipsel-nacl/include/c++/v1");
+    addSystemInclude(DriverArgs, CC1Args, P.str());
----------------
it seems wrong to include a C++ path here; I don't think this function is C++ specific?
And in any case, is this intended to override -nobuiltininc?

================
Comment at: lib/Driver/Tools.cpp:1184
@@ +1183,3 @@
+
+  if (Triple.getOS() == llvm::Triple::NaCl) {
+    CmdArgs.push_back("-mllvm");
----------------
this part may not be upstreamable since the LLVM part that implements it won't be upstreamable.

================
Comment at: lib/Driver/Tools.cpp:8327
@@ +8326,3 @@
+        // without '-lnacl' it prefers symbols from libpthread.a over libnacl.a,
+        // which is not a desired behaviour here.
+        // See https://sourceware.org/ml/binutils/2015-03/msg00034.html
----------------
which symbols are causing trouble here, and how? can you give some more detail?

I wasn't aware of the difference, but I have an unrelated change that may be affected by it.

http://reviews.llvm.org/D10739

EMAIL PREFERENCES
  http://reviews.llvm.org/settings/panel/emailpreferences/






More information about the cfe-commits mailing list