[clang] [clang-refactor] Add Matcher Edit refactoring rule (PR #123782)

2025-04-24 Thread Сергеев Игнатий via cfe-commits
IgnatSergeev wrote: @bartlettroscoe, > So is this really transforming the AST? Well yeah, you are right, it replaces it with text, but what i ment, is that it supports replacements with EditGenerators, which are really expressive (a few words about them https://clang.llvm.org/docs/ClangTrans

[clang] [clang-refactor] Add Matcher Edit refactoring rule (PR #123782)

2025-04-22 Thread Roscoe A. Bartlett via cfe-commits
=?utf-8?b?0JjQs9C90LDRgiDQodC10YDQsw=?=, =?utf-8?b?0JjQs9C90LDRgiDQodC10YDQsw=?Message-ID: In-Reply-To: bartlettroscoe wrote: @IgnatSergeev, thanks for the reply! > You were right about the amount of refactorings that LibTool could support > And actually AST transformations are already suppor

[clang] [clang-refactor] Add Matcher Edit refactoring rule (PR #123782)

2025-04-19 Thread Сергеев Игнатий via cfe-commits
IgnatSergeev wrote: @bartlettroscoe, sorry for not responding, at some point, i forgot checking my github notifications You were right about the amount of refactorings that LibTool could support And actually AST transformations are already supported in clang-transformer tool and its library, s

[clang] [clang-refactor] Add Matcher Edit refactoring rule (PR #123782)

2025-01-27 Thread Roscoe A. Bartlett via cfe-commits
=?utf-8?b?0JjQs9C90LDRgiDQodC10YDQsw=?=, =?utf-8?b?0JjQs9C90LDRgiDQodC10YDQsw=?Message-ID: In-Reply-To: bartlettroscoe wrote: @IgnatSergeev, thanks for the detailed information on the clang-refactor testing infrastructure. We will try to run these tests and dig in deeper to see how the testi

[clang] [clang-refactor] Add Matcher Edit refactoring rule (PR #123782)

2025-01-26 Thread Сергеев Игнатий via cfe-commits
IgnatSergeev wrote: According to PR workflows, llvm uses external CI platform (buildkite). `buildkite/github-pull-requests/linux-linux-x64` workflow seems like the one which builds and tests changes https://github.com/llvm/llvm-project/pull/123782 ___

[clang] [clang-refactor] Add Matcher Edit refactoring rule (PR #123782)

2025-01-26 Thread Сергеев Игнатий via cfe-commits
IgnatSergeev wrote: > @IgnatSergeev, thanks for posting this PR! > > First question, where is this new code tested? > > Do you know the status of automated testing for this LLVM refactoring support > software and the `clang-refactor` tool? > > According to: > > * https://llvm.org/docs/Testin

[clang] [clang-refactor] Add Matcher Edit refactoring rule (PR #123782)

2025-01-21 Thread via cfe-commits
___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [clang-refactor] Add Matcher Edit refactoring rule (PR #123782)

2025-01-21 Thread via cfe-commits
=?utf-8?b?0JjQs9C90LDRgiDQodC10YDQsw=?=, =?utf-8?b?0JjQs9C90LDRgiDQodC10YDQsw=?Message-ID: In-Reply-To: github-actions[bot] wrote: Thank you for submitting a Pull Request (PR) to the LLVM Project! This PR will be automatically labeled and the relevant teams will be notified. If you wish to,

[clang] [clang-refactor] Add Matcher Edit refactoring rule (PR #123782)

2025-01-21 Thread Сергеев Игнатий via cfe-commits
https://github.com/IgnatSergeev created https://github.com/llvm/llvm-project/pull/123782 Modified capabilities of refactoring engine and clang-refactor tool: - Added source location argument for clang-refactor tool - Added corresponding source location requirement, that requests SourceLocation -