From dbd3c85825ad59896292ac7d326fe1985768f1e3 Mon Sep 17 00:00:00 2001 From: Cameron Zwarich Date: Thu, 7 Jul 2011 21:03:28 +0000 Subject: [PATCH] r134634 causes a failure on MultiSource/Benchmarks/Olden/bh with TEST=nightly, so roll it out. git-svn-id: https://llvm.org/svn/llvm-project/cfe/trunk@134638 91177308-0d34-0410-b5e6-96231b3b80d8 --- include/clang/AST/Decl.h | 2 -- lib/AST/ASTContext.cpp | 2 +- lib/AST/Decl.cpp | 26 -------------------------- lib/CodeGen/CodeGenModule.cpp | 17 ++--------------- test/CodeGen/inline.c | 7 ------- 5 files changed, 3 insertions(+), 51 deletions(-) diff --git a/include/clang/AST/Decl.h b/include/clang/AST/Decl.h index b7ea7a1be1..143915b4b5 100644 --- a/include/clang/AST/Decl.h +++ b/include/clang/AST/Decl.h @@ -1773,8 +1773,6 @@ public: bool isInlined() const; bool isInlineDefinitionExternallyVisible() const; - - bool doesDeclarationForceExternallyVisibleDefinition() const; /// isOverloadedOperator - Whether this function declaration /// represents an C++ overloaded operator, e.g., "operator+". diff --git a/lib/AST/ASTContext.cpp b/lib/AST/ASTContext.cpp index f2bd47dfa5..e2fa4e504a 100644 --- a/lib/AST/ASTContext.cpp +++ b/lib/AST/ASTContext.cpp @@ -6376,7 +6376,7 @@ bool ASTContext::DeclMustBeEmitted(const Decl *D) { if (const FunctionDecl *FD = dyn_cast(D)) { // Forward declarations aren't required. if (!FD->doesThisDeclarationHaveABody()) - return FD->doesDeclarationForceExternallyVisibleDefinition(); + return false; // Constructors and destructors are required. if (FD->hasAttr() || FD->hasAttr()) diff --git a/lib/AST/Decl.cpp b/lib/AST/Decl.cpp index 9feec9d82a..9b507cfc5e 100644 --- a/lib/AST/Decl.cpp +++ b/lib/AST/Decl.cpp @@ -1762,32 +1762,6 @@ bool FunctionDecl::isInlined() const { return false; } -/// \brief For a function declaration in C or C++, determine whether this -/// declaration causes the definition to be externally visible. -/// -/// Determines whether this is the first non-inline redeclaration of an inline -/// function in a language where "inline" does not normally require an -/// externally visible definition. -bool FunctionDecl::doesDeclarationForceExternallyVisibleDefinition() const { - assert(!doesThisDeclarationHaveABody() && - "Must have a declaration without a body."); - - ASTContext &Context = getASTContext(); - - // In C99 mode, a function may have an inline definition (causing it to - // be deferred) then redeclared later. As a special case, "extern inline" - // is not required to produce an external symbol. - if (Context.getLangOptions().GNUInline || !Context.getLangOptions().C99 || - Context.getLangOptions().CPlusPlus) - return false; - if (getLinkage() != ExternalLinkage || isInlineSpecified()) - return false; - const FunctionDecl *InlineDefinition = 0; - if (hasBody(InlineDefinition)) - return InlineDefinition->isInlineDefinitionExternallyVisible(); - return false; -} - /// \brief For an inline function definition in C or C++, determine whether the /// definition will be externally visible. /// diff --git a/lib/CodeGen/CodeGenModule.cpp b/lib/CodeGen/CodeGenModule.cpp index b2905299e4..ddef39726f 100644 --- a/lib/CodeGen/CodeGenModule.cpp +++ b/lib/CodeGen/CodeGenModule.cpp @@ -597,7 +597,7 @@ void CodeGenModule::EmitLLVMUsed() { void CodeGenModule::EmitDeferred() { // Emit code for any potentially referenced deferred decls. Since a // previously unused static decl may become used during the generation of code - // for a static function, iterate until no changes are made. + // for a static function, iterate until no changes are made. while (!DeferredDeclsToEmit.empty() || !DeferredVTables.empty()) { if (!DeferredVTables.empty()) { @@ -740,21 +740,8 @@ void CodeGenModule::EmitGlobal(GlobalDecl GD) { } // Forward declarations are emitted lazily on first use. - if (!FD->doesThisDeclarationHaveABody()) { - if (!FD->doesDeclarationForceExternallyVisibleDefinition()) - return; - - const FunctionDecl *InlineDefinition = 0; - FD->getBody(InlineDefinition); - - llvm::StringRef MangledName = getMangledName(GD); - llvm::StringMap::iterator DDI = - DeferredDecls.find(MangledName); - if (DDI != DeferredDecls.end()) - DeferredDecls.erase(DDI); - EmitGlobalDefinition(InlineDefinition); + if (!FD->doesThisDeclarationHaveABody()) return; - } } else { const VarDecl *VD = cast(Global); assert(VD->isFileVarDecl() && "Cannot emit local var decl as global."); diff --git a/test/CodeGen/inline.c b/test/CodeGen/inline.c index cbc428cc22..96f9c5c9d4 100644 --- a/test/CodeGen/inline.c +++ b/test/CodeGen/inline.c @@ -12,7 +12,6 @@ // RUN: grep "define void @test3()" %t // RUN: grep "define available_externally i32 @test4" %t // RUN: grep "define available_externally i32 @test5" %t -// RUN: grep "define i32 @test6" %t // RUN: echo "\nC99 tests:" // RUN: %clang %s -O1 -emit-llvm -S -o %t -std=c99 @@ -28,7 +27,6 @@ // RUN: grep "define void @test3" %t // RUN: grep "define available_externally i32 @test4" %t // RUN: grep "define available_externally i32 @test5" %t -// RUN: grep "define i32 @test6" %t // RUN: echo "\nC++ tests:" // RUN: %clang -x c++ %s -O1 -emit-llvm -S -o %t -std=c++98 @@ -86,8 +84,3 @@ extern __inline int __attribute__ ((__gnu_inline__)) test5(void) } void test_test5() { test5(); } - -// PR10233 - -__inline int test6() { return 0; } -extern int test6(); -- 2.50.1