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

_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to