[PATCH] D19551: Handle Windows drive letters and ".." for --reproduce.

[Davide Italiano via llvm-commits]

davide added inline comments.

================
Comment at: ELF/Driver.cpp:106
@@ +105,3 @@
+//  1. The result will never escape from S. Therefore, all ".."
+//     are removed from T before concatenati them.
+//  2. Windows drive letters are removed from T before concatenation.
----------------
concatenating.

================
Comment at: ELF/Driver.cpp:258
@@ -243,6 +257,3 @@
 
-static void dumpLinkerInvocation(ArrayRef<const char *> Args) {
-  std::error_code EC = sys::fs::create_directories(Config->Reproduce,
-    false /* IgnoreExisting */);
-  if (EC) {
-    error(EC, "--reproduce: can't create directory");
+static void logCommandline(ArrayRef<const char *> Args) {
+  if (std::error_code EC = sys::fs::create_directories(
----------------
I prefer dumpLinkerInvocation still.

================
Comment at: ELF/Driver.cpp:272
@@ -257,3 +271,3 @@
   OS << Args[0];
-  for (unsigned I = 1, E = Args.size(); I < E; ++I)
+  for (size_t I = 1, E = Args.size(); I < E; ++I)
     OS << " " << Args[I];
----------------
We use unsigned everywhere else in the code base, try to be consistent or change all the other occurrences.


http://reviews.llvm.org/D19551