[llvm] [llvm][unittests] Put human-readable names on TargetParserTests. NFC (PR #80749)

Jon Roelofs via llvm-commits llvm-commits at lists.llvm.org
Tue Feb 6 08:43:33 PST 2024


================
@@ -163,6 +163,15 @@ template <typename T> struct ARMCPUTestParams {
     return os;
   }
 
+  static std::string
+  PrintToStringParamName(const testing::TestParamInfo<ARMCPUTestParams<T>>& Info) {
+    std::string Name = Info.param.CPUName.str();
+    for (char &C : Name)
+      if (!std::isalnum(C))
----------------
jroelofs wrote:

A bunch of them have `-`s in them, and it is a requirement that gtest test names only have alphanumeric characters in them: https://github.com/google/googletest/blob/main/docs/advanced.md#specifying-names-for-value-parameterized-test-parameters

https://github.com/llvm/llvm-project/pull/80749


More information about the llvm-commits mailing list