[PATCH] D117650: [llvm][vfs] Implement in-memory symlinks

Duncan P. N. Exon Smith via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Jan 21 15:05:28 PST 2022


dexonsmith added a comment.

In D117650#3262196 <https://reviews.llvm.org/D117650#3262196>, @sammccall wrote:

> In D117650#3262164 <https://reviews.llvm.org/D117650#3262164>, @sammccall wrote:
>
>> (IIRC my previous concerns weren't about VFS->VFS symlinks, but rather ability to "resolve" a VFS file to a non-virtual file in order to open it through native APIs. It's possible I misunderstood that proposal of course)
>
> Oops, I just remembered about D97288 <https://reviews.llvm.org/D97288>, which is probably what you meant. Yeah, this change is much less expensive because it doesn't affect (other) VFS implementers nor consumers.

Yes, I think that's what I was remembering!


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D117650



More information about the llvm-commits mailing list