philnik added a comment.
In D149573#4310009 <https://reviews.llvm.org/D149573#4310009>, @codemzs wrote:
> My change to libcxxabi/test/test_demangle.pass.cpp (last file in the change)
> is only one line i.e {"_ZNK5clang4Type16isArithmeticTypeERNS_10ASTContextE",
> "clang::Type::isArithmeticType(clang::ASTContext&) const"}, but running
> clang-format on it changes bunch of lines.
Please don't clang-format the test then. There is no need to do it, and the
changes look rather confusing. I don't really understand why you add it at all
TBH. This doesn't look like it tests anything from this patch.
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D149573/new/
https://reviews.llvm.org/D149573
_______________________________________________
cfe-commits mailing list
[email protected]
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
