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 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
