================ @@ -1012,11 +1013,18 @@ TEST(TargetParserTest, getARMCPUForArch) { TEST(TargetParserTest, ARMPrintSupportedExtensions) { std::string expected = "All available -march extensions for ARM\n\n" - "\tcrc\n\tcrypto\n\tsha2"; + " Name Description\n" + " crc This is a long dummy description\n" + " crypto This is a long dummy description\n" + " sha2 This is a long dummy description\n"; + + llvm::StringMap<llvm::StringRef> DummyMap; + for (const auto &E : llvm::ARM::ARCHExtNames) + DummyMap.insert({E.Name, "This is a long dummy description"}); ---------------- DavidSpickett wrote:
This is ok but what I meant specifically was, add a dummy description for just one extension. Then these tests are covering the case where we do and don't have an extension. https://github.com/llvm/llvm-project/pull/66715 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits