[PATCH] D58890: Modules: Rename MemoryBufferCache to InMemoryModuleCache

[Duncan P. N. Exon Smith via Phabricator via cfe-commits]

dexonsmith added a comment.

In D58890#1417465 <https://reviews.llvm.org/D58890#1417465>, @bruno wrote:

> `InMemoryModuleCache` seems like a way more appropriate name here. Also thanks for improving some of the comments.
>
> > Because of the move to Serialization we can no longer abuse the Preprocessor to forward it to the ASTReader. Besides the rename and file move, that means Preprocessor::Preprocessor has one fewer parameter and ASTReader::ASTReader has one more.
>
> Are there any C api bits that expose preprocessor stuff and also need updates?


`ninja check-clang` passes... is there anything else I should be testing?


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

https://reviews.llvm.org/D58890