[NFC][Clang] Remove unused function `CodeGenModule::addDefaultFunctionDefinitionAttributes`

[jmmartinez]

Summary

This patch deletes the unused addDefaultFunctionDefinitionAttributes(llvm::Function); function,
while it still keeps void addDefaultFunctionDefinitionAttributes(llvm::AttrBuilder &attrs); which is being used.

clang/lib/CodeGen/CGCall.cpp

void CodeGenModule::getDefaultFunctionAttributes(StringRef Name,
   getTrivialDefaultFunctionAttributes(Name, HasOptnone, AttrOnCallSite,
                                       FuncAttrs);
   // If we're just getting the default, get the default values for mergeable
   // attributes.
   if (!AttrOnCallSite)
     addMergableDefaultFunctionAttributes(CodeGenOpts, FuncAttrs);
 }
 
-void CodeGenModule::addDefaultFunctionDefinitionAttributes(llvm::Function &F) {
-  llvm::AttrBuilder FuncAttrs(F.getContext());
-  getDefaultFunctionAttributes(F.getName(), F.hasOptNone(),
-                               /* AttrOnCallSite = */ false, FuncAttrs);
-  // TODO: call GetCPUAndFeaturesAttributes?
-  F.addFnAttrs(FuncAttrs);
-}
-
 /// Apply default attributes to \p F, accounting for merge semantics of
 /// attributes that should not overwrite existing attributes.
 void CodeGenModule::mergeDefaultFunctionDefinitionAttributes(
     llvm::Function &F, bool WillInternalize) {
   ::mergeDefaultFunctionDefinitionAttributes(F, getCodeGenOpts(), getLangOpts(),
                                              getTarget().getTargetOpts(),
                                              WillInternalize);
 }

clang/lib/CodeGen/CodeGenModule.h

public:
   /// unsafe-fp-math attrs as tantamount to unsafe-fp-math=false, and then LLVM
   /// will propagate unsafe-fp-math=false up to every transitive caller of a
   /// function in the bitcode library!
   ///
   /// With the exception of fast-math attrs, this will only make the attributes
   /// on the function more conservative.  But it's unsafe to call this on a
   /// function which relies on particular fast-math attributes for correctness.
   /// It's up to you to ensure that this is safe.
-  void addDefaultFunctionDefinitionAttributes(llvm::Function &F);
   void mergeDefaultFunctionDefinitionAttributes(llvm::Function &F,
                                                 bool WillInternalize);
 
   /// Like the overload taking a `Function &`, but intended specifically
   /// for frontends that want to build on Clang's target-configuration logic.
   void addDefaultFunctionDefinitionAttributes(llvm::AttrBuilder &attrs);
 
   StringRef getMangledName(GlobalDecl GD);