[PATCH] D114029: [clang][NFC] Inclusive terms: rename AccessDeclContextSanity to AccessDeclContextCheck
This revision was landed with ongoing or failed builds. This revision was automatically updated to reflect the committed changes. Closed by commit rG5162b558d8c0: [clang][NFC] Inclusive terms: rename AccessDeclContextSanity to… (authored by ZarkoCA). Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D114029/new/ https://reviews.llvm.org/D114029 Files: clang/include/clang/AST/DeclBase.h clang/lib/AST/DeclBase.cpp lldb/source/Plugins/TypeSystem/Clang/TypeSystemClang.cpp Index: lldb/source/Plugins/TypeSystem/Clang/TypeSystemClang.cpp === --- lldb/source/Plugins/TypeSystem/Clang/TypeSystemClang.cpp +++ lldb/source/Plugins/TypeSystem/Clang/TypeSystemClang.cpp @@ -91,7 +91,7 @@ assert(decl && "VerifyDecl called with nullptr?"); #ifndef NDEBUG // We don't care about the actual access value here but only want to trigger - // that Clang calls its internal Decl::AccessDeclContextSanity check. + // that Clang calls its internal Decl::AccessDeclContextCheck validation. decl->getAccess(); #endif } Index: clang/lib/AST/DeclBase.cpp === --- clang/lib/AST/DeclBase.cpp +++ clang/lib/AST/DeclBase.cpp @@ -964,7 +964,7 @@ return {}; } -bool Decl::AccessDeclContextSanity() const { +bool Decl::AccessDeclContextCheck() const { #ifndef NDEBUG // Suppress this check if any of the following hold: // 1. this is the translation unit (and thus has no parent) Index: clang/include/clang/AST/DeclBase.h === --- clang/include/clang/AST/DeclBase.h +++ clang/include/clang/AST/DeclBase.h @@ -352,7 +352,7 @@ DeclContext *Parent, std::size_t Extra = 0); private: - bool AccessDeclContextSanity() const; + bool AccessDeclContextCheck() const; /// Get the module ownership kind to use for a local lexical child of \p DC, /// which may be either a local or (rarely) an imported declaration. @@ -472,11 +472,11 @@ void setAccess(AccessSpecifier AS) { Access = AS; -assert(AccessDeclContextSanity()); +assert(AccessDeclContextCheck()); } AccessSpecifier getAccess() const { -assert(AccessDeclContextSanity()); +assert(AccessDeclContextCheck()); return AccessSpecifier(Access); } Index: lldb/source/Plugins/TypeSystem/Clang/TypeSystemClang.cpp === --- lldb/source/Plugins/TypeSystem/Clang/TypeSystemClang.cpp +++ lldb/source/Plugins/TypeSystem/Clang/TypeSystemClang.cpp @@ -91,7 +91,7 @@ assert(decl && "VerifyDecl called with nullptr?"); #ifndef NDEBUG // We don't care about the actual access value here but only want to trigger - // that Clang calls its internal Decl::AccessDeclContextSanity check. + // that Clang calls its internal Decl::AccessDeclContextCheck validation. decl->getAccess(); #endif } Index: clang/lib/AST/DeclBase.cpp === --- clang/lib/AST/DeclBase.cpp +++ clang/lib/AST/DeclBase.cpp @@ -964,7 +964,7 @@ return {}; } -bool Decl::AccessDeclContextSanity() const { +bool Decl::AccessDeclContextCheck() const { #ifndef NDEBUG // Suppress this check if any of the following hold: // 1. this is the translation unit (and thus has no parent) Index: clang/include/clang/AST/DeclBase.h === --- clang/include/clang/AST/DeclBase.h +++ clang/include/clang/AST/DeclBase.h @@ -352,7 +352,7 @@ DeclContext *Parent, std::size_t Extra = 0); private: - bool AccessDeclContextSanity() const; + bool AccessDeclContextCheck() const; /// Get the module ownership kind to use for a local lexical child of \p DC, /// which may be either a local or (rarely) an imported declaration. @@ -472,11 +472,11 @@ void setAccess(AccessSpecifier AS) { Access = AS; -assert(AccessDeclContextSanity()); +assert(AccessDeclContextCheck()); } AccessSpecifier getAccess() const { -assert(AccessDeclContextSanity()); +assert(AccessDeclContextCheck()); return AccessSpecifier(Access); } ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
[PATCH] D114029: [clang][NFC] Inclusive terms: rename AccessDeclContextSanity to AccessDeclContextCheck
quinnp accepted this revision. quinnp 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/D114029/new/ https://reviews.llvm.org/D114029 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
[PATCH] D114029: [clang][NFC] Inclusive terms: rename AccessDeclContextSanity to AccessDeclContextCheck
ZarkoCA created this revision. ZarkoCA added reviewers: klimek, akyrtzi, quinnp. ZarkoCA added a project: clang. ZarkoCA requested review of this revision. Herald added subscribers: lldb-commits, cfe-commits. Herald added a project: LLDB. Rename function to more inclusive name. Repository: rG LLVM Github Monorepo https://reviews.llvm.org/D114029 Files: clang/include/clang/AST/DeclBase.h clang/lib/AST/DeclBase.cpp lldb/source/Plugins/TypeSystem/Clang/TypeSystemClang.cpp Index: lldb/source/Plugins/TypeSystem/Clang/TypeSystemClang.cpp === --- lldb/source/Plugins/TypeSystem/Clang/TypeSystemClang.cpp +++ lldb/source/Plugins/TypeSystem/Clang/TypeSystemClang.cpp @@ -91,7 +91,7 @@ assert(decl && "VerifyDecl called with nullptr?"); #ifndef NDEBUG // We don't care about the actual access value here but only want to trigger - // that Clang calls its internal Decl::AccessDeclContextSanity check. + // that Clang calls its internal Decl::AccessDeclContextCheck validation. decl->getAccess(); #endif } Index: clang/lib/AST/DeclBase.cpp === --- clang/lib/AST/DeclBase.cpp +++ clang/lib/AST/DeclBase.cpp @@ -964,7 +964,7 @@ return {}; } -bool Decl::AccessDeclContextSanity() const { +bool Decl::AccessDeclContextCheck() const { #ifndef NDEBUG // Suppress this check if any of the following hold: // 1. this is the translation unit (and thus has no parent) Index: clang/include/clang/AST/DeclBase.h === --- clang/include/clang/AST/DeclBase.h +++ clang/include/clang/AST/DeclBase.h @@ -352,7 +352,7 @@ DeclContext *Parent, std::size_t Extra = 0); private: - bool AccessDeclContextSanity() const; + bool AccessDeclContextCheck() const; /// Get the module ownership kind to use for a local lexical child of \p DC, /// which may be either a local or (rarely) an imported declaration. @@ -472,11 +472,11 @@ void setAccess(AccessSpecifier AS) { Access = AS; -assert(AccessDeclContextSanity()); +assert(AccessDeclContextCheck()); } AccessSpecifier getAccess() const { -assert(AccessDeclContextSanity()); +assert(AccessDeclContextCheck()); return AccessSpecifier(Access); } Index: lldb/source/Plugins/TypeSystem/Clang/TypeSystemClang.cpp === --- lldb/source/Plugins/TypeSystem/Clang/TypeSystemClang.cpp +++ lldb/source/Plugins/TypeSystem/Clang/TypeSystemClang.cpp @@ -91,7 +91,7 @@ assert(decl && "VerifyDecl called with nullptr?"); #ifndef NDEBUG // We don't care about the actual access value here but only want to trigger - // that Clang calls its internal Decl::AccessDeclContextSanity check. + // that Clang calls its internal Decl::AccessDeclContextCheck validation. decl->getAccess(); #endif } Index: clang/lib/AST/DeclBase.cpp === --- clang/lib/AST/DeclBase.cpp +++ clang/lib/AST/DeclBase.cpp @@ -964,7 +964,7 @@ return {}; } -bool Decl::AccessDeclContextSanity() const { +bool Decl::AccessDeclContextCheck() const { #ifndef NDEBUG // Suppress this check if any of the following hold: // 1. this is the translation unit (and thus has no parent) Index: clang/include/clang/AST/DeclBase.h === --- clang/include/clang/AST/DeclBase.h +++ clang/include/clang/AST/DeclBase.h @@ -352,7 +352,7 @@ DeclContext *Parent, std::size_t Extra = 0); private: - bool AccessDeclContextSanity() const; + bool AccessDeclContextCheck() const; /// Get the module ownership kind to use for a local lexical child of \p DC, /// which may be either a local or (rarely) an imported declaration. @@ -472,11 +472,11 @@ void setAccess(AccessSpecifier AS) { Access = AS; -assert(AccessDeclContextSanity()); +assert(AccessDeclContextCheck()); } AccessSpecifier getAccess() const { -assert(AccessDeclContextSanity()); +assert(AccessDeclContextCheck()); return AccessSpecifier(Access); } ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits