aaronpuchert marked 3 inline comments as done. aaronpuchert added inline comments.
================ Comment at: lib/Analysis/ThreadSafety.cpp:960-961 + FSet.removeLock(FactMan, !UnderCp); + FSet.addLock(FactMan, llvm::make_unique<LockableFactEntry>(UnderCp, LK, + RelockLoc)); + } ---------------- aaron.ballman wrote: > aaronpuchert wrote: > > Looks a bit weird, but `clang-format` told me to do that. > This line looks correct to me, but the `else` statement above doesn't match > our usual formatting rules, so I'm worried you're running clang-format in a > way that's not picking up the LLVM coding style options. You're right about the `else`, not sure what happened there. Repository: rC Clang https://reviews.llvm.org/D49885 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
