[PATCH] D26288: Deduplicate replacements by FileEntry instead of file names.
    Alexander Shaposhnikov via cfe-commits 
    cfe-commits at lists.llvm.org
       
    Thu Nov  3 14:19:27 PDT 2016
    
    
  
alexshap added inline comments.
================
Comment at: lib/Tooling/Core/Replacement.cpp:578
+      continue;
+    ProcessedFileEntries.insert(FE);
+    Result[Entry.first] = std::move(Entry.second);
----------------
(saves 2 lines of code) (although not particularly important) 
  if (ProcessedFileEntries.insert(FE).second)
     Result[Entry.first] = std::move(Entry.second);
https://reviews.llvm.org/D26288
    
    
More information about the cfe-commits
mailing list