[PATCH] D151607: [AST] Remove unused function removeLocalCVRQualifiers

2023-06-06 Thread Aaron Ballman via Phabricator via cfe-commits
aaron.ballman added a comment. In D151607#4383132 , @kazu wrote: > In D151607#4380757 , @erichkeane > wrote: > >> Since this is part of the public API, do we need to publish/alert folks of >> this? This could

[PATCH] D151607: [AST] Remove unused function removeLocalCVRQualifiers

2023-05-30 Thread Kazu Hirata via Phabricator via cfe-commits
kazu added a comment. In D151607#4380757 , @erichkeane wrote: > Since this is part of the public API, do we need to publish/alert folks of > this? This could perhaps be in use by someone using libclang/etc, right? I don't think we guarantee the API

[PATCH] D151607: [AST] Remove unused function removeLocalCVRQualifiers

2023-05-30 Thread Erich Keane via Phabricator via cfe-commits
erichkeane added a comment. Since this is part of the public API, do we need to publish/alert folks of this? This could perhaps be in use by someone using libclang/etc, right? Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D151607/new/

[PATCH] D151607: [AST] Remove unused function removeLocalCVRQualifiers

2023-05-27 Thread Kazu Hirata via Phabricator via cfe-commits
This revision was landed with ongoing or failed builds. This revision was automatically updated to reflect the committed changes. Closed by commit rGe396c375a108: [AST] Remove unused function removeLocalCVRQualifiers (authored by kazu). Repository: rG LLVM Github Monorepo CHANGES SINCE LAST

[PATCH] D151607: [AST] Remove unused function removeLocalCVRQualifiers

2023-05-27 Thread Corentin Jabot via Phabricator via cfe-commits
cor3ntin accepted this revision. cor3ntin added a comment. This revision is now accepted and ready to land. LGTM Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D151607/new/ https://reviews.llvm.org/D151607

[PATCH] D151607: [AST] Remove unused function removeLocalCVRQualifiers

2023-05-26 Thread Kazu Hirata via Phabricator via cfe-commits
kazu created this revision. Herald added a project: All. kazu requested review of this revision. Herald added a project: clang. Herald added a subscriber: cfe-commits. The last use was removed by: commit 0eb06cb3aa2700508c20da28f22ff91e7b82a436 Author: Roy Jacobson Date: Tue Mar 14