grimar accepted this revision. grimar added a comment. This revision is now accepted and ready to land.
LGTM. It worth wainting for a second approvement and/or other comments to verify that people are happy with doing this for `clang-format`. ================ Comment at: clang/unittests/Format/FormatTest.cpp:16062 FS.addFile("/d/test.cpp", 0, llvm::MemoryBuffer::getMemBuffer("int i;"))); auto Style7 = getStyle("file", "/d/.clang-format", "LLVM", "", &FS); ASSERT_FALSE((bool)Style7); ---------------- nit: since you have `Style7b`, this perhaps should be `Style7a`. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D86137/new/ https://reviews.llvm.org/D86137 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits