[PATCH] D139774: [libclang] Add API to set temporary directory location

[Aaron Ballman via Phabricator via llvm-commits]

aaron.ballman added a comment.

In D139774#4079249 <https://reviews.llvm.org/D139774#4079249>, @vedgy wrote:

> Now that a consensus has been reached that this revision is to be discarded, can we please resume the discussion of which alternative should be implemented in the replacement revision?

Based on the discussion here, I think the preference is to modify `FileSystemOptions` to store an override for the temp directory, and updating APIs so that the temp directory can be specified by a new CIndex constructor function. (If anyone disagrees with that as the approach we'd like to take, please speak up!)


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D139774