LGTM with a nit.

================
Comment at: lib/Driver/ToolChains.cpp:1739
@@ +1738,3 @@
+        if (StringRef(M.includeSuffix()).startswith("/uclibc"))
+          Dirs.push_back(
+              (InstallDir + "/../../../../sysroot/uclibc/usr/include").str());
----------------
Since you are calling str() anyway, it might be cleaner to construct it with

  String S = InstallDir + "/../../../../sysroot/";
  if (...)
    S +=
   else
     S +=

http://reviews.llvm.org/D4790



_______________________________________________
cfe-commits mailing list
[email protected]
http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits

Reply via email to