[BuildLibCalls][NFC] Remove redundant attribute list from emitCalloc

[yurai007]

Summary

Additionally with this patch aligned DSE which is the only user of emitCalloc.

[yurai007]

Rely on D103451.

[xbolva00]

Thanks!

[yurai007]

@xbolva00: Could you please also take a look on D103009? D103009 is required for this change and now should be ready to land as well.

[yurai007]

Rebased.

[yurai007]

Rebased again.

llvm/include/llvm/Transforms/Utils/BuildLibCalls.h

namespace llvm {
   Value *emitFWrite(Value *Ptr, Value *Size, Value *File, IRBuilderBase &B,
                     const DataLayout &DL, const TargetLibraryInfo *TLI);
 
   /// Emit a call to the malloc function.
   Value *emitMalloc(Value *Num, IRBuilderBase &B, const DataLayout &DL,
                     const TargetLibraryInfo *TLI);
 
   /// Emit a call to the calloc function.
-  Value *emitCalloc(Value *Num, Value *Size, const AttributeList &Attrs,
-                    IRBuilderBase &B, const TargetLibraryInfo &TLI);
+  Value *emitCalloc(Value *Num, Value *Size, IRBuilderBase &B,
+                    const TargetLibraryInfo &TLI);
 }
 
 #endif

llvm/lib/Transforms/Scalar/DeadStoreElimination.cpp

if (StoredConstant && StoredConstant->isNullValue()) {
             return false;
           if (Malloc->getOperand(0) == MemSet->getLength()) {
             if (DT.dominates(DefUOInst, cast<Instruction>(MemSet)) &&
                 memoryIsNotModifiedBetween(Malloc, MemSet, BatchAA, DL, &DT)) {
               IRBuilder<> IRB(Malloc);
               const auto &DL = Malloc->getModule()->getDataLayout();
               auto *SizeType =
                   DL.getIntPtrType(IRB.GetInsertBlock()->getContext());
-              AttributeList EmptyList;
-              if (auto *Calloc = emitCalloc(ConstantInt::get(SizeType, 1),
-                                            Malloc->getArgOperand(0), EmptyList,
-                                            IRB, TLI)) {
+              if (auto *Calloc =
+                      emitCalloc(ConstantInt::get(SizeType, 1),
+                                 Malloc->getArgOperand(0), IRB, TLI)) {
                 Malloc->replaceAllUsesWith(Calloc);
                 Malloc->eraseFromParent();
                 return true;
               }
               return false;
             }
           }
         }

llvm/lib/Transforms/Utils/BuildLibCalls.cpp

Value *llvm::emitMalloc(Value *Num, IRBuilderBase &B, const DataLayout &DL,
 
   if (const Function *F =
           dyn_cast<Function>(Malloc.getCallee()->stripPointerCasts()))
     CI->setCallingConv(F->getCallingConv());
 
   return CI;
 }
 
-Value *llvm::emitCalloc(Value *Num, Value *Size, const AttributeList &Attrs,
-                        IRBuilderBase &B, const TargetLibraryInfo &TLI) {
+Value *llvm::emitCalloc(Value *Num, Value *Size, IRBuilderBase &B,
+                        const TargetLibraryInfo &TLI) {
   if (!TLI.has(LibFunc_calloc))
     return nullptr;
 
   Module *M = B.GetInsertBlock()->getModule();
   StringRef CallocName = TLI.getName(LibFunc_calloc);
   const DataLayout &DL = M->getDataLayout();
   IntegerType *PtrType = DL.getIntPtrType((B.GetInsertBlock()->getContext()));
-  FunctionCallee Calloc = M->getOrInsertFunction(
-      CallocName, Attrs, B.getInt8PtrTy(), PtrType, PtrType);
+  FunctionCallee Calloc =
+      M->getOrInsertFunction(CallocName, B.getInt8PtrTy(), PtrType, PtrType);
   inferLibFuncAttributes(M, CallocName, TLI);
   CallInst *CI = B.CreateCall(Calloc, {Num, Size}, CallocName);
 
   if (const auto *F =
           dyn_cast<Function>(Calloc.getCallee()->stripPointerCasts()))
     CI->setCallingConv(F->getCallingConv());
 
   return CI;
 }