[clang] [clang-tools-extra] [clang-query] Allow for trailing comma in matchers (PR #148018)

2025-07-12 Thread Remy Farley via cfe-commits
one-d-wide wrote: I don't think this is much of an issue. Once you attempt to interpret a query with trailing comma in it as C++ code, a compiler (or even an LSP) will usually emit a (mostly) unambiguous error, which should be trivial to correct. And that's it. Existing queries, including ones

[clang] [clang-tools-extra] [clang-query] Allow for trailing comma in matchers (PR #148018)

2025-07-12 Thread Remy Farley via cfe-commits
https://github.com/one-d-wide updated https://github.com/llvm/llvm-project/pull/148018 >From b35f9ebdd2e345449c2cea8df8d6731cde33dcb3 Mon Sep 17 00:00:00 2001 From: "Remy D. Farley" Date: Sat, 12 Jul 2025 11:53:58 + Subject: [PATCH] [clang-query] Allow for trailing comma in matchers --- c

[clang] [clang-tools-extra] [clang-query] Allow for trailing comma in matchers (PR #148018)

2025-07-10 Thread Remy Farley via cfe-commits
https://github.com/one-d-wide created https://github.com/llvm/llvm-project/pull/148018 Allow AST matches in clang-query to have a trailing comma at the end of matcher arguments. Makes it nicer to work with queries that span multiple lines. So, for example, the following is possible: ```clang-