[clang] [ASTMatchers] AST matcher support for ObjC pointers (PR #117021)

2024-12-03 Thread LLVM Continuous Integration via cfe-commits
llvm-ci wrote: LLVM Buildbot has detected a new failure on builder `clang-armv8-quick` running on `linaro-clang-armv8-quick` while building `clang` at step 5 "ninja check 1". Full details are available at: https://lab.llvm.org/buildbot/#/builders/154/builds/8421 Here is the relevant piece of

[clang] [ASTMatchers] AST matcher support for ObjC pointers (PR #117021)

2024-12-03 Thread Rashmi Mudduluru via cfe-commits
https://github.com/t-rasmud closed https://github.com/llvm/llvm-project/pull/117021 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [ASTMatchers] AST matcher support for ObjC pointers (PR #117021)

2024-12-03 Thread Rashmi Mudduluru via cfe-commits
https://github.com/t-rasmud updated https://github.com/llvm/llvm-project/pull/117021 >From 8cbc2754a4525d35dac720c283efaea1167deb9d Mon Sep 17 00:00:00 2001 From: Rashmi Mudduluru Date: Tue, 19 Nov 2024 14:50:24 -0800 Subject: [PATCH] [ASTMatchers] AST matcher support for ObjC pointers Add `Ob

[clang] [ASTMatchers] AST matcher support for ObjC pointers (PR #117021)

2024-12-03 Thread Rashmi Mudduluru via cfe-commits
https://github.com/t-rasmud updated https://github.com/llvm/llvm-project/pull/117021 >From f624a90eb8c893d5de16d0426c2884ca03ccbd4c Mon Sep 17 00:00:00 2001 From: Rashmi Mudduluru Date: Tue, 19 Nov 2024 14:50:24 -0800 Subject: [PATCH] [ASTMatchers] AST matcher support for ObjC pointers Add `Ob

[clang] [ASTMatchers] AST matcher support for ObjC pointers (PR #117021)

2024-12-03 Thread Rashmi Mudduluru via cfe-commits
t-rasmud wrote: Thanks for the review! https://github.com/llvm/llvm-project/pull/117021 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [ASTMatchers] AST matcher support for ObjC pointers (PR #117021)

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

[clang] [ASTMatchers] AST matcher support for ObjC pointers (PR #117021)

2024-12-02 Thread Rashmi Mudduluru via cfe-commits
t-rasmud wrote: Friendly @ping for approval/feedback. https://github.com/llvm/llvm-project/pull/117021 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [ASTMatchers] AST matcher support for ObjC pointers (PR #117021)

2024-11-22 Thread Rashmi Mudduluru via cfe-commits
https://github.com/t-rasmud updated https://github.com/llvm/llvm-project/pull/117021 >From 87ba2a10ca7435fdf6b5c47d2c1c97c1e188cfcd Mon Sep 17 00:00:00 2001 From: Rashmi Mudduluru Date: Tue, 19 Nov 2024 14:50:24 -0800 Subject: [PATCH 1/3] [ASTMatchers] AST matcher support for ObjC pointers Add

[clang] [ASTMatchers] AST matcher support for ObjC pointers (PR #117021)

2024-11-22 Thread Rashmi Mudduluru via cfe-commits
t-rasmud wrote: > The changes LGTM as far as they go, but can you add details to the patch > summary about why these should be exposed? (We typically only add to the AST > matchers when there's an in-tree need for the functionality, so are there > checks being updated to make use of these new

[clang] [ASTMatchers] AST matcher support for ObjC pointers (PR #117021)

2024-11-22 Thread Rashmi Mudduluru via cfe-commits
https://github.com/t-rasmud edited https://github.com/llvm/llvm-project/pull/117021 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [ASTMatchers] AST matcher support for ObjC pointers (PR #117021)

2024-11-22 Thread Aaron Ballman via cfe-commits
https://github.com/AaronBallman commented: The changes LGTM as far as they go, but can you add details to the patch summary about why these should be exposed? (We typically only add to the AST matchers when there's an in-tree need for the functionality, so are there checks being updated to mak

[clang] [ASTMatchers] AST matcher support for ObjC pointers (PR #117021)

2024-11-20 Thread Rashmi Mudduluru via cfe-commits
https://github.com/t-rasmud created https://github.com/llvm/llvm-project/pull/117021 Add `ObjCInterfaceDecl` to the list of types supported by the `hasType` and `hasDeclaration` matchers, `ObjCObjectPointerType` to the list of types supported by `pointee`. >From 87ba2a10ca7435fdf6b5c47d2c1c97

[clang] [ASTMatchers] AST matcher support for ObjC pointers (PR #117021)

2024-11-20 Thread Ziqing Luo via cfe-commits
https://github.com/ziqingluo-90 approved this pull request. Neat! https://github.com/llvm/llvm-project/pull/117021 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [ASTMatchers] AST matcher support for ObjC pointers (PR #117021)

2024-11-20 Thread via cfe-commits
llvmbot wrote: @llvm/pr-subscribers-clang Author: Rashmi Mudduluru (t-rasmud) Changes Add `ObjCInterfaceDecl` to the list of types supported by the `hasType` and `hasDeclaration` matchers, `ObjCObjectPointerType` to the list of types supported by `pointee`. --- Full diff: https://github

[clang] [ASTMatchers] AST matcher support for ObjC pointers (PR #117021)

2024-11-20 Thread via cfe-commits
github-actions[bot] wrote: :warning: C/C++ code formatter, clang-format found issues in your code. :warning: You can test this locally with the following command: ``bash git-clang-format --diff def22f4e718daa74c2d0c03a32e32d4913a46278 87ba2a10ca7435fdf6b5c47d2c1c97c1e188cfcd --e

[clang] [ASTMatchers] AST matcher support for ObjC pointers (PR #117021)

2024-11-20 Thread Rashmi Mudduluru via cfe-commits
https://github.com/t-rasmud updated https://github.com/llvm/llvm-project/pull/117021 >From 87ba2a10ca7435fdf6b5c47d2c1c97c1e188cfcd Mon Sep 17 00:00:00 2001 From: Rashmi Mudduluru Date: Tue, 19 Nov 2024 14:50:24 -0800 Subject: [PATCH 1/2] [ASTMatchers] AST matcher support for ObjC pointers Add