[PATCH] D116174: [clang] support relative roots to vfs overlays

Richard Howell via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Fri Jan 14 07:36:59 PST 2022


rmaz added a comment.

In D116174#3242087 <https://reviews.llvm.org/D116174#3242087>, @benlangmuir wrote:

> Each VFS may have its own working directory, so it could be surprising if we use the OS working directory instead of that.  This is complicated by the fact the VFS working directory may not be set yet during parsing the yaml (I haven't checked).  I'm not really sure what to recommend here. If we do change this, we should document the new behaviour in the doc comment for RedirectingFileSystem though.

I don't believe the working directory will be set on the VFS at this point, no. If you're happy with the change then i'll update the header docs to describe this behaviour, unless there is another way to achieve using relative paths for root entries in the overlay yaml?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D116174



More information about the cfe-commits mailing list