[llvm] r184630 - Create the file with the right permissions instead of setting it afterwards.

Rafael Espindola rafael.espindola at gmail.com
Fri Jun 21 19:34:24 PDT 2013


Author: rafael
Date: Fri Jun 21 21:34:24 2013
New Revision: 184630

URL: http://llvm.org/viewvc/llvm-project?rev=184630&view=rev
Log:
Create the file with the right permissions instead of setting it afterwards.

Removes the last use of PathV1.h in llvm-ar.

Modified:
    llvm/trunk/tools/llvm-ar/ArchiveWriter.cpp

Modified: llvm/trunk/tools/llvm-ar/ArchiveWriter.cpp
URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/tools/llvm-ar/ArchiveWriter.cpp?rev=184630&r1=184629&r2=184630&view=diff
==============================================================================
--- llvm/trunk/tools/llvm-ar/ArchiveWriter.cpp (original)
+++ llvm/trunk/tools/llvm-ar/ArchiveWriter.cpp Fri Jun 21 21:34:24 2013
@@ -18,7 +18,6 @@
 #include "llvm/IR/Module.h"
 #include "llvm/Support/FileSystem.h"
 #include "llvm/Support/MemoryBuffer.h"
-#include "llvm/Support/PathV1.h"
 #include "llvm/Support/Process.h"
 #include "llvm/Support/Signals.h"
 #include "llvm/Support/system_error.h"
@@ -269,7 +268,7 @@ bool Archive::writeToDisk(bool TruncateN
   int TmpArchiveFD;
   SmallString<128> TmpArchive;
   error_code EC = sys::fs::unique_file("temp-archive-%%%%%%%.a", TmpArchiveFD,
-                                       TmpArchive);
+                                       TmpArchive, true, 0666);
   if (EC)
     return true;
 
@@ -305,12 +304,5 @@ bool Archive::writeToDisk(bool TruncateN
     return true;
   }
 
-  // Set correct read and write permissions after temporary file is moved
-  // to final destination path.
-  if (sys::Path(archPath).makeReadableOnDisk(ErrMsg))
-    return true;
-  if (sys::Path(archPath).makeWriteableOnDisk(ErrMsg))
-    return true;
-
   return false;
 }





More information about the llvm-commits mailing list