[clang] [clang] fix hasAnyBase not binding submatchers (PR #67939)

2023-10-06 Thread Aaron Ballman via cfe-commits
https://github.com/AaronBallman approved this pull request. LGTM! https://github.com/llvm/llvm-project/pull/67939 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [clang] fix hasAnyBase not binding submatchers (PR #67939)

2023-10-04 Thread Shafik Yaghmour via cfe-commits
shafik wrote: I think you should also add `[ASTMatcher]` to the subject after `[Clang]` at least that is what past PRs seem to do. https://github.com/llvm/llvm-project/pull/67939 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.ll

[clang] [clang] fix hasAnyBase not binding submatchers (PR #67939)

2023-10-03 Thread via cfe-commits
5chmidti wrote: Added a comment to the release notes https://github.com/llvm/llvm-project/pull/67939 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [clang] fix hasAnyBase not binding submatchers (PR #67939)

2023-10-03 Thread via cfe-commits
https://github.com/5chmidti updated https://github.com/llvm/llvm-project/pull/67939 >From d0ac0cdb0f9ccbe78ec236932ca9b9a85768ecf2 Mon Sep 17 00:00:00 2001 From: Julian Schmidt <44101708+5chmi...@users.noreply.github.com> Date: Mon, 2 Oct 2023 01:34:24 +0200 Subject: [PATCH] [clang] fix hasAnyBa

[clang] [clang] fix hasAnyBase not binding submatchers (PR #67939)

2023-10-01 Thread via cfe-commits
llvmbot wrote: @llvm/pr-subscribers-clang Changes The BoundNodesTreeBuilder used in the BaseSpecMatcher was the original and was reset to its original state if a match occurred. The matcher now uses the local copy in the inner matcher. Fixes #65421 --- Full diff: https://github.com/llvm/

[clang] [clang] fix hasAnyBase not binding submatchers (PR #67939)

2023-10-01 Thread via cfe-commits
https://github.com/5chmidti created https://github.com/llvm/llvm-project/pull/67939 The BoundNodesTreeBuilder used in the BaseSpecMatcher was the original and was reset to its original state if a match occurred. The matcher now uses the local copy in the inner matcher. Fixes #65421 >From 8a57