[PATCH] D37554: [libclang] Allow crash recovery with LIBCLANG_NOTHREADS
Ilya Biryukov via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Wed Sep 27 08:53:37 PDT 2017
ilya-biryukov added a comment.
I think your change makes sense, but maybe asking for a better description.
It appears you:
- Enabled crash recovery for some libclang operations on a calling thread even when `LIBCLANG_NOTHREAD` is specified. Previously it would only run under crash recovery if `LIBCLANG_NOTHREAD` is not set.
- Moved handling of `LIBCLANG_NOTHREAD` env variable into `RunSafely` from its call sites.
That said, I am not familiar with the code you're changing, so can't really LGTM this. Hopefully, someone else can do that.
https://reviews.llvm.org/D37554
More information about the cfe-commits
mailing list