[PATCH] D132867: [Clang] Use virtual FS in processing config files
    Serge Pavlov via Phabricator via llvm-commits 
    llvm-commits at lists.llvm.org
       
    Tue Sep  6 10:23:22 PDT 2022
    
    
  
sepavloff added a comment.
In D132867#3771004 <https://reviews.llvm.org/D132867#3771004>, @sammccall wrote:
>> Strictly speaking this is incorrect behavior because file system object must be used for all operations on files, according to documentation.
>
> I don't think this is a strong argument against documenting the change (is that what we're disagreeing about?)
No problem to add documentation. I added a release note, if you mean that.
> - in practice many users who rely on these details work it out by trial and error
Absolutely!
Repository:
  rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D132867/new/
https://reviews.llvm.org/D132867
    
    
More information about the llvm-commits
mailing list