[PATCH] D70769: [Support] add vfs support for ExpandResponseFiles

James Henderson via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Wed Nov 27 03:16:58 PST 2019


jhenderson added a comment.

In D70769#1761428 <https://reviews.llvm.org/D70769#1761428>, @lh123 wrote:

> In D70769#1761394 <https://reviews.llvm.org/D70769#1761394>, @jhenderson wrote:
>
> > Are there any instances where we DON'T want to get the real file system? If not, could the `*llvm::vfs::getRealFileSystem()` call be put inside `cl::ExpandResponseFiles`?
>
>
> This patch is part of D70222 <https://reviews.llvm.org/D70222>.
>
>   This is for using InMemoryFileSystem in unit tests.


Okay, that makes sense. Perhaps we should introduce a new overload of `ExpandResponseFiles` that allows specifying a different file system then, and having the current version call into that one, specifying the getRealFileSystem call? I'm not overly keen on having yet another apparently boiler-plate piece of functionality required in every single call of `ExpandResponseFiles`.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D70769





More information about the cfe-commits mailing list