[PATCH] D78058: option to write files to memory instead of disk

2021-01-26 Thread Duncan P. N. Exon Smith via Phabricator via cfe-commits
dexonsmith added a comment. In D78058#2481050 , @dexonsmith wrote: > In D78058#2480411 , @dexonsmith > wrote: > >> In D78058#2471735 , @sammccall >> wrote: >> >>> @dexonsmi

[PATCH] D78058: option to write files to memory instead of disk

2021-01-05 Thread Duncan P. N. Exon Smith via Phabricator via cfe-commits
dexonsmith added a comment. In D78058#2480411 , @dexonsmith wrote: > In D78058#2471735 , @sammccall wrote: > >> @dexonsmith thanks for sharing! >> Some initial thoughts since abstracting outputs is something we're s

[PATCH] D78058: option to write files to memory instead of disk

2021-01-05 Thread Duncan P. N. Exon Smith via Phabricator via cfe-commits
dexonsmith added a comment. In D78058#2471735 , @sammccall wrote: > @dexonsmith thanks for sharing! > Some initial thoughts since abstracting outputs is something we're starting > to care about too... Thanks for looking. > This doesn't appear to be an e

[PATCH] D78058: option to write files to memory instead of disk

2020-12-26 Thread Sam McCall via Phabricator via cfe-commits
sammccall added a comment. @dexonsmith thanks for sharing! Some initial thoughts since abstracting outputs is something we're starting to care about too... This doesn't appear to be an extension point - you can write to an InMemoryFS or to real disk, but not to anything else. If we're going to

[PATCH] D78058: option to write files to memory instead of disk

2020-12-23 Thread Duncan P. N. Exon Smith via Phabricator via cfe-commits
dexonsmith added a comment. In D78058#2464262 , @dexonsmith wrote: > I'll reply here once I've posted the RFC and patch (as I said, I'm hoping > next week) so you can take a look. I don't quite have my patch and RFC ready (and I may not until I'm back fr

[PATCH] D78058: option to write files to memory instead of disk

2020-12-18 Thread Duncan P. N. Exon Smith via Phabricator via cfe-commits
dexonsmith added subscribers: erik.pilkington, arphaman, Bigcheese. dexonsmith added a comment. I think this is important; thanks for working on it! I'm sorry I missed it before... coincidentally, I'm working in a similar problem space right now (likely I'll post a patch with an RFC next week) b

[PATCH] D78058: option to write files to memory instead of disk

2020-12-17 Thread Sam McCall via Phabricator via cfe-commits
sammccall added a comment. It's great that you're looking at this! Clang tooling in general and clangd in particular a lot of use of VFS (particularly when deployed inside google), so I've got a fair bit of interest/experience in these interfaces. And we'd also looked at virtualizing module cac

[PATCH] D78058: option to write files to memory instead of disk

2020-12-02 Thread Marc Rasi via Phabricator via cfe-commits
marcrasi added a comment. Ping on this. Could anyone take a look? Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D78058/new/ https://reviews.llvm.org/D78058 ___ cfe-commits mailing list cfe-commits@lists.l

[PATCH] D78058: option to write files to memory instead of disk

2020-04-13 Thread Marc Rasi via Phabricator via cfe-commits
marcrasi created this revision. Herald added subscribers: cfe-commits, jfb, mgorny. Herald added a project: clang. marcrasi edited the summary of this revision. This allows clients to invoke clang without writing anything to disk. I'm not sure if this is the right design, so I'm sending this patc