vsapsai marked an inline comment as done.
vsapsai added inline comments.

================
Comment at: clang/lib/Serialization/ModuleManager.cpp:211
-    if (!getModuleCache().tryToDropPCM(NewModule->FileName))
-      FileMgr.invalidateCache(NewModule->File);
     return OutOfDate;
----------------
dexonsmith wrote:
> This is the only caller of FileManager::invalidateCache.  I suggest deleting 
> it in a follow-up, in order to reduce complexity in the FileManager.
Good suggestion. Agree it will be better to keep the FileManager simpler.


Repository:
  rL LLVM

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D66907/new/

https://reviews.llvm.org/D66907



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to