[libcxx-commits] [libcxx] [libc++] Remove workaround for clang-tidy 16 in the test suite setup code (PR #69035)

via libcxx-commits libcxx-commits at lists.llvm.org
Fri Oct 13 17:16:29 PDT 2023


llvmbot wrote:


<!--LLVM PR SUMMARY COMMENT-->

@llvm/pr-subscribers-libcxx

Author: Louis Dionne (ldionne)

<details>
<summary>Changes</summary>

We have moved to clang-tidy 17 now, so the workaround shouldn't be necessary.

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


1 Files Affected:

- (modified) libcxx/utils/libcxx/test/params.py (-5) 


``````````diff
diff --git a/libcxx/utils/libcxx/test/params.py b/libcxx/utils/libcxx/test/params.py
index 456794b9b1cce95..0c043cc6e55ccee 100644
--- a/libcxx/utils/libcxx/test/params.py
+++ b/libcxx/utils/libcxx/test/params.py
@@ -77,11 +77,6 @@
 
 
 def getStdFlag(cfg, std):
-    # TODO(LLVM-17) Remove this clang-tidy-16 work-around
-    if std == "c++23":
-        std = "c++2b"
-    if hasCompileFlag(cfg, "-std=" + std):
-        return "-std=" + std
     # TODO(LLVM-19) Remove the fallbacks needed for Clang 16.
     fallbacks = {
         "c++23": "c++2b",

``````````

</details>


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


More information about the libcxx-commits mailing list