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

Aaron Ballman via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Wed Jan 25 13:04:19 PST 2023


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



More information about the cfe-commits mailing list