[PATCH] D97288: Added `Follow` parameter to llvm::vfs::FileSystem::status()

Sam McCall via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Feb 23 07:13:03 PST 2021


sammccall added a comment.

How strong is the need for this?
This adds complexity to a widely implemented and used interface, and the combination of virtual + default parameters can be at least a little confusing.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D97288



More information about the llvm-commits mailing list