jkorous-apple added a comment. LGTM otherwise.
================ Comment at: tools/libclang/CIndexer.h:45 + std::string ToolchainPath; + llvm::sys::SmartMutex<false> Mutex; + ---------------- I am just wondering - since we anticipate multi-threaded usage, shouldn't we synchronize access to all member variables mutable via public class interface? Specifically - shouldn't we use one mutex per every mutating/reading method group? - setInvocationEmissionPath(), getInvocationEmissionPath() - setCXGlobalOptFlags(), isOptEnabled() - getClangResourcesPath() Repository: rC Clang https://reviews.llvm.org/D40527 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits