From 2dfa957f120244ddc0238b548282f67d89f1a832 Mon Sep 17 00:00:00 2001 From: Chuanqi Xu Date: Mon, 20 Feb 2023 15:37:11 +0800 Subject: [PATCH] [NFC] Remove the unused parameter in Sema::PushGlobalModuleFragment The `IsImplicit` parameter should be removed since it is not used now. --- clang/include/clang/Sema/Sema.h | 2 +- clang/lib/Sema/SemaDeclCXX.cpp | 2 +- clang/lib/Sema/SemaExprCXX.cpp | 2 +- clang/lib/Sema/SemaModule.cpp | 8 +++----- 4 files changed, 6 insertions(+), 8 deletions(-) diff --git a/clang/include/clang/Sema/Sema.h b/clang/include/clang/Sema/Sema.h index a6b1102..176d92a 100644 --- a/clang/include/clang/Sema/Sema.h +++ b/clang/include/clang/Sema/Sema.h @@ -2294,7 +2294,7 @@ private: } /// Enter the scope of the global module. - Module *PushGlobalModuleFragment(SourceLocation BeginLoc, bool IsImplicit); + Module *PushGlobalModuleFragment(SourceLocation BeginLoc); /// Leave the scope of the global module. void PopGlobalModuleFragment(); diff --git a/clang/lib/Sema/SemaDeclCXX.cpp b/clang/lib/Sema/SemaDeclCXX.cpp index 7bae979..6ec0eac 100644 --- a/clang/lib/Sema/SemaDeclCXX.cpp +++ b/clang/lib/Sema/SemaDeclCXX.cpp @@ -16363,7 +16363,7 @@ Decl *Sema::ActOnStartLinkageSpecification(Scope *S, SourceLocation ExternLoc, /// need to attach it again. if (getLangOpts().CPlusPlusModules && isCurrentModulePurview()) { Module *GlobalModule = - PushGlobalModuleFragment(ExternLoc, /*IsImplicit=*/true); + PushGlobalModuleFragment(ExternLoc); /// According to [module.reach]p3.2, /// The declaration in global module fragment is reachable if it is not /// discarded. And the discarded declaration should be deleted. So it diff --git a/clang/lib/Sema/SemaExprCXX.cpp b/clang/lib/Sema/SemaExprCXX.cpp index eb9a3e6..81d2c31 100644 --- a/clang/lib/Sema/SemaExprCXX.cpp +++ b/clang/lib/Sema/SemaExprCXX.cpp @@ -2979,7 +2979,7 @@ void Sema::DeclareGlobalNewDelete() { // functions are replaceable ([new.delete]); these are attached to the // global module ([module.unit]). if (getLangOpts().CPlusPlusModules && getCurrentModule()) - PushGlobalModuleFragment(SourceLocation(), /*IsImplicit=*/true); + PushGlobalModuleFragment(SourceLocation()); // C++ [basic.std.dynamic]p2: // [...] The following allocation and deallocation functions (18.4) are diff --git a/clang/lib/Sema/SemaModule.cpp b/clang/lib/Sema/SemaModule.cpp index f03d982..59891ec 100644 --- a/clang/lib/Sema/SemaModule.cpp +++ b/clang/lib/Sema/SemaModule.cpp @@ -74,10 +74,9 @@ static std::string stringFromPath(ModuleIdPath Path) { Sema::DeclGroupPtrTy Sema::ActOnGlobalModuleFragmentDecl(SourceLocation ModuleLoc) { - // We start in the global module; all those declarations are implicitly - // module-private (though they do not have module linkage). + // We start in the global module; Module *GlobalModule = - PushGlobalModuleFragment(ModuleLoc, /*IsImplicit=*/false); + PushGlobalModuleFragment(ModuleLoc); // All declarations created from now on are owned by the global module. auto *TU = Context.getTranslationUnitDecl(); @@ -956,8 +955,7 @@ Decl *Sema::ActOnFinishExportDecl(Scope *S, Decl *D, SourceLocation RBraceLoc) { return D; } -Module *Sema::PushGlobalModuleFragment(SourceLocation BeginLoc, - bool IsImplicit) { +Module *Sema::PushGlobalModuleFragment(SourceLocation BeginLoc) { // We shouldn't create new global module fragment if there is already // one. if (!GlobalModuleFragment) { -- 2.7.4