[PATCH] D66859: [index-while-buildling] FSUtil

[Dmitri Gribenko via Phabricator via llvm-commits]

gribozavr accepted this revision.
gribozavr added a comment.
This revision is now accepted and ready to land.

In D66859#1649868 <https://reviews.llvm.org/D66859#1649868>, @jkorous wrote:

> Hmm, actually if we replace `Buffer` in the interface with a `llvm::function_ref<void(llvm::raw_ostream &)>` callback it would cover couple other existing uses of `createUniqueFile`.


That would make a lot of sense if you're going to do the cleanup and refactor those callers to use new API. Good idea!

However, I think just from the API simplicity perspective, it would make sense to have two overloaded APIs if calling this function with a precomputed buffer is common.


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

https://reviews.llvm.org/D66859