jyknight added inline comments.

================
Comment at: lib/Driver/ToolChains.cpp:4319
@@ +4318,3 @@
+  // This contains libc, libg (a superset of libc), libm, libstdc++, libssp.
+  SmallString<128> LibDir(GCCInstallation.getParentLibPath());
+  if (Triple.getArch() == llvm::Triple::sparcel)
----------------
Should check GCCInstallation.isValid() before doing any of this.

================
Comment at: lib/Driver/ToolChains.cpp:4324
@@ +4323,3 @@
+    llvm::sys::path::append(LibDir, "../sparc-myriad-elf/lib");
+  if (getVFS().exists(LibDir))
+    getFilePaths().push_back(LibDir.str());
----------------
addPathIfExists instead of if/push_back.


http://reviews.llvm.org/D14754



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to