[PATCH] D139774: [libclang] Add API to set temporary directory location
Igor Kushnir via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Wed Feb 1 08:01:27 PST 2023
vedgy added a comment.
Perhaps the struct should contain the value of `CINDEX_VERSION_MINOR` instead of the `sizeof`? This way, libclang can change the meaning of a struct member without changing the size of the struct. For example, replace `PreambleStoragePath` with `TemporaryDirectoryPath`. Such a change could even be quiet, because setting the more general temporary directory path would likely be welcome or at least acceptable to the API users.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D139774/new/
https://reviews.llvm.org/D139774
More information about the llvm-commits
mailing list