DavidSpickett added inline comments.
================ Comment at: llvm/unittests/Support/TargetParserTest.cpp:833 + } + } pass &= AArch64::getCPUAttr(AK).equals(CPUAttr); ---------------- I assume this was left in from debugging, if not it should be its own change. (considering this file is already using gtest, you could refactor to ASSERT_EQ the flags and get the messages for free) Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D93022/new/ https://reviews.llvm.org/D93022 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
