llvmbot wrote:
<!--LLVM PR SUMMARY COMMENT--> @llvm/pr-subscribers-clang Author: Boaz Brickner (bricknerb) <details> <summary>Changes</summary> Also, remove the non-const `ASTUnit::getASTContext()` since it's no longer necessary. This would make it similar to `Decl::getAstContext() const` for a more consistent and flexible API. --- Full diff: https://github.com/llvm/llvm-project/pull/130096.diff 1 Files Affected: - (modified) clang/include/clang/Frontend/ASTUnit.h (+1-2) ``````````diff diff --git a/clang/include/clang/Frontend/ASTUnit.h b/clang/include/clang/Frontend/ASTUnit.h index 1f98c6ab328ba..bd55c8b627941 100644 --- a/clang/include/clang/Frontend/ASTUnit.h +++ b/clang/include/clang/Frontend/ASTUnit.h @@ -439,8 +439,7 @@ class ASTUnit { Preprocessor &getPreprocessor() { return *PP; } std::shared_ptr<Preprocessor> getPreprocessorPtr() const { return PP; } - const ASTContext &getASTContext() const { return *Ctx; } - ASTContext &getASTContext() { return *Ctx; } + ASTContext &getASTContext() const { return *Ctx; } void setASTContext(ASTContext *ctx) { Ctx = ctx; } void setPreprocessor(std::shared_ptr<Preprocessor> pp); `````````` </details> https://github.com/llvm/llvm-project/pull/130096 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
