[PATCH] D52620: Added Support for StatOnly Files in VFS.

Kirill Bobyrev via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Fri Sep 28 02:54:54 PDT 2018


kbobyrev added inline comments.


================
Comment at: lib/Basic/VirtualFileSystem.cpp:2097
 void YAMLVFSWriter::write(llvm::raw_ostream &OS) {
-  llvm::sort(Mappings, [](const YAMLVFSEntry &LHS, const YAMLVFSEntry &RHS) {
+  llvm::sort(Mappings.begin(), Mappings.end(),
+             [](const YAMLVFSEntry &LHS, const YAMLVFSEntry &RHS) {
----------------
This used `llvm::sort(Container &&C, Compare Comp)` before, I would think that the range-based API is the preferred one. There's also a cleanup patch migrating STL-like calls to idiomatic LLVM's STL Extension API: D52576.


Repository:
  rC Clang

https://reviews.llvm.org/D52620





More information about the cfe-commits mailing list