]> granicus.if.org Git - clang/commitdiff
Add a fast path to CodeGenModule::getMangledName for almost all C functions,
authorChris Lattner <sabre@nondot.org>
Sat, 21 Mar 2009 06:31:09 +0000 (06:31 +0000)
committerChris Lattner <sabre@nondot.org>
Sat, 21 Mar 2009 06:31:09 +0000 (06:31 +0000)
speeding up the testcase in PR3810 by 60%.

git-svn-id: https://llvm.org/svn/llvm-project/cfe/trunk@67431 91177308-0d34-0410-b5e6-96231b3b80d8

lib/CodeGen/CodeGenModule.cpp

index cc460c595937c1c7d0ced9470e304d98fe57d742..b03513a0147ec929e8da243cdfff2c31272115fe 100644 (file)
@@ -156,8 +156,8 @@ static void setGlobalVisibility(llvm::GlobalValue *GV,
 /// \brief Retrieves the mangled name for the given declaration.
 ///
 /// If the given declaration requires a mangled name, returns an
-/// IdentifierInfo* containing the mangled name. Otherwise, returns
-/// the name of the declaration as an identifier.
+/// const char* containing the mangled name.  Otherwise, returns
+/// the unmangled name.
 ///
 /// FIXME: Returning an IdentifierInfo* here is a total hack. We
 /// really need some kind of string abstraction that either stores a
@@ -171,6 +171,12 @@ static void setGlobalVisibility(llvm::GlobalValue *GV,
 /// caching mangled names. However, we should fix the problem above
 /// first.
 const char *CodeGenModule::getMangledName(const NamedDecl *ND) {
+  // In C, functions with no attributes never need to be mangled. Fastpath them.
+  if (!getLangOptions().CPlusPlus && !ND->hasAttrs()) {
+    assert(ND->getIdentifier() && "Attempt to mangle unnamed decl.");
+    return ND->getIdentifier()->getName();
+  }
+    
   llvm::SmallString<256> Name;
   llvm::raw_svector_ostream Out(Name);
   if (!mangleName(ND, Context, Out)) {