LGTM ================ Comment at: unittests/Support/LockFileManagerTest.cpp:48 @@ +47,2 @@ + +} // end anon namespace ---------------- ultra nit: i think "end anonymous namespace" is more common http://llvm-reviews.chandlerc.com/D1304 BRANCH master ARCANIST PROJECT llvm