[clang] [Clang][C++26] Implement Pack Indexing (P2662R3). (PR #72644)

2024-01-27 Thread Jie Fu via cfe-commits
DamonFool wrote: lldb may also need to be fixed? ``` llvm-project/lldb/source/Plugins/TypeSystem/Clang/TypeSystemClang.cpp:4074:11: error: enumeration value 'PackIndexing' not handled in switch [-Werror,-Wswitch] switch (qual_type->getTypeClass()) { ^ llvm-pro

[clang] [clang] SyntaxWarning: invalid escape sequence '\s' with Python3.12 (PR #78036)

2024-01-13 Thread Jie Fu via cfe-commits
https://github.com/DamonFool created https://github.com/llvm/llvm-project/pull/78036 The following SyntaxWarning was observed with Python3.12 . ``` llvm-project/clang/tools/libclang/linker-script-to-export-list.py:9: SyntaxWarning: invalid escape sequence '\s' m = re.search("^\s+(clang_[^;]+)

[clang] [clang] SyntaxWarning: invalid escape sequence '\s' with Python3.12 (PR #78036)

2024-01-13 Thread Jie Fu via cfe-commits
https://github.com/DamonFool updated https://github.com/llvm/llvm-project/pull/78036 >From 57fa67bf86bea09b9c6011b8c9ec6f69fe11852f Mon Sep 17 00:00:00 2001 From: Jie Fu Date: Sat, 13 Jan 2024 21:51:36 +0800 Subject: [PATCH 1/2] [clang] SyntaxWarning: invalid escape sequence '\s' with Python3.

[clang] [clang] SyntaxWarning: invalid escape sequence '\s' with Python3.12 (PR #78036)

2024-01-13 Thread Jie Fu via cfe-commits
@@ -6,6 +6,6 @@ output_file = open(sys.argv[2], "w") for line in input_file: -m = re.search("^\s+(clang_[^;]+)", line) +m = re.search("^\\s+(clang_[^;]+)", line) DamonFool wrote: > Making it a raw string is a marginally cleaner fix Yes, agree. Update

[clang] [clang] SyntaxWarning: invalid escape sequence '\s' with Python3.12 (PR #78036)

2024-01-14 Thread Jie Fu via cfe-commits
https://github.com/DamonFool closed https://github.com/llvm/llvm-project/pull/78036 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [clang] SyntaxWarning: invalid escape sequence '\s' with Python3.12 (PR #78036)

2024-01-14 Thread Jie Fu via cfe-commits
DamonFool wrote: Thanks @MaskRay and @cor3ntin for the review. https://github.com/llvm/llvm-project/pull/78036 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] hurd: Fix build with -Werror,-Wswitch (PR #78520)

2024-01-17 Thread Jie Fu via cfe-commits
DamonFool wrote: > Is there a reason you put this first? Otherwise IMO it is slightly neater to > put it last. This bug had been fixed here: ``` commit 1b60ddd920e0caadfa85cc7013b559d6453d7e3e Author: Jie Fu Date: Thu Jan 18 07:34:09 2024 +0800 [Hurd] Fix -Wswitch in Hurd::getDynamicLi