pwnall added a comment. In D52578#1307145 <https://reviews.llvm.org/D52578#1307145>, @aaronpuchert wrote:
> To be clear, I'm not a big fan of this change myself, I just wanted to see if > it was feasible. My personal opinion, as I wrote in the bug report, is that > scoped releasing of mutexes is taking RAII a step too far. I'm putting this > on ice for now until we're reached a state where it looks a bit less crazy. I > hope @pwnall can live with that, since Clang 8 will not come out soon anyway. If it makes a difference, Chrome (loosely) tracks Clang trunk <https://cs.chromium.org/chromium/src/tools/clang/scripts/update.py?q=CLANG_REVISION&l=30>. We wouldn't wait for a stable release to adopt the new code. Repository: rC Clang CHANGES SINCE LAST ACTION https://reviews.llvm.org/D52578/new/ https://reviews.llvm.org/D52578 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits