[libcxx-commits] [libcxxabi] [libcxxabi][test][NFC] Turn off clang-format for demangler test-case array (PR #73503)

via libcxx-commits libcxx-commits at lists.llvm.org
Mon Nov 27 03:19:02 PST 2023


llvmbot wrote:


<!--LLVM PR SUMMARY COMMENT-->

@llvm/pr-subscribers-libcxxabi

Author: Michael Buch (Michael137)

<details>
<summary>Changes</summary>

Adding test-cases to the `cases` array causes `git clang-format` to split the strings of many of the existing test-cases, making them harder to read/work with in most cases.

This patch disables `clang-format` for the `cases` array so it doesn't catch anyone off-guard in the future.

---
Full diff: https://github.com/llvm/llvm-project/pull/73503.diff


1 Files Affected:

- (modified) libcxxabi/test/test_demangle.pass.cpp (+2) 


``````````diff
diff --git a/libcxxabi/test/test_demangle.pass.cpp b/libcxxabi/test/test_demangle.pass.cpp
index 77741a952850ab9..ff56d8772869c19 100644
--- a/libcxxabi/test/test_demangle.pass.cpp
+++ b/libcxxabi/test/test_demangle.pass.cpp
@@ -32,6 +32,7 @@
 // Is long double fp128?
 #define LDBL_FP128 (__LDBL_MANT_DIG__ == 113)
 
+// clang-format off
 const char* cases[][2] =
 {
     {"_Z1A", "A"},
@@ -30175,6 +30176,7 @@ const char* cases[][2] =
     {"_Z2f5IPiEvu16__remove_pointerIT_E", "void f5<int*>(__remove_pointer(int*))"},
     {"_Z2f5IiEvu14__remove_cvrefIT_E", "void f5<int>(__remove_cvref(int))"},
 };
+// clang-format on
 
 const unsigned N = sizeof(cases) / sizeof(cases[0]);
 

``````````

</details>


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


More information about the libcxx-commits mailing list