[WIP][FileCollector] FileCollector::Mutex

[jkorous]

Summary

IIUC we should synchronize read and write access to VFSWriter.

It almost feels like we might want to keep FileCollector thread-unsafe and move the responsibility (and the mutex) to client code.

[JDevlieghere]

llvm/include/llvm/Support/FileCollector.h

72

Yep. How about "Synchronizes the file mapping."?

llvm/include/llvm/Support/FileCollector.h

 
 protected:
   void addFileImpl(StringRef SrcPath);
 
   llvm::vfs::directory_iterator
   addDirectoryImpl(const llvm::Twine &Dir,
                    IntrusiveRefCntPtr<vfs::FileSystem> FS, std::error_code &EC);
 
-  /// Synchronizes adding files.
+  /// Synchronizes access to Seen, VFSWriter and SymlinkMap.
   std::mutex Mutex;
 
   /// The root directory where files are copied.
   std::string Root;
 
   /// The root directory where the VFS overlay lives.
   std::string OverlayRoot;
 

llvm/lib/Support/FileCollector.cpp

copyAccessAndModificationTime(StringRef Filename,
 
   if (auto EC = sys::Process::SafelyCloseFileDescriptor(FD))
     return EC;
 
   return {};
 }
 
 std::error_code FileCollector::copyFiles(bool StopOnError) {
+  std::lock_guard<std::mutex> lock(Mutex);
+
   for (auto &entry : VFSWriter.getMappings()) {
     // Create directory tree.
     if (std::error_code EC =
             sys::fs::create_directories(sys::path::parent_path(entry.RPath),
                                         /*IgnoreExisting=*/true)) {
       if (StopOnError)
         return EC;
     }