Use getCalledFunctionName() throughout the codebase.

tsymalla · tsymalla · commit 793cde7169e1 · 2025-02-21T13:28:09.000+01:00
diff --git a/llvm/include/llvm/Analysis/VectorUtils.h b/llvm/include/llvm/Analysis/VectorUtils.h
@@ -41,7 +41,7 @@ class VFDatabase {
   /// a vector Function ABI.
   static void getVFABIMappings(const CallInst &CI,
                                SmallVectorImpl<VFInfo> &Mappings) {
-    const auto ScalarName = CI.getCalledFunctionName();
+    const std::optional<StringRef> ScalarName = CI.getCalledFunctionName();
     if (!ScalarName.has_value())
       return;
 
diff --git a/llvm/lib/Analysis/IRSimilarityIdentifier.cpp b/llvm/lib/Analysis/IRSimilarityIdentifier.cpp
@@ -153,7 +153,7 @@ void IRInstructionData::setCalleeName(bool MatchByName) {
   }
 
   if (!CI->isIndirectCall() && MatchByName)
-    CalleeName = CI->getCalledFunction()->getName().str();
+    CalleeName = CI->getCalledFunctionName()->str();
 }
 
 void IRInstructionData::setPHIPredecessors(
diff --git a/llvm/lib/Analysis/KernelInfo.cpp b/llvm/lib/Analysis/KernelInfo.cpp
@@ -159,7 +159,7 @@ static void remarkFlatAddrspaceAccess(OptimizationRemarkEmitter &ORE,
     R << "in ";
     identifyFunction(R, Caller);
     if (const IntrinsicInst *II = dyn_cast<IntrinsicInst>(&Inst)) {
-      R << ", '" << II->getCalledFunction()->getName() << "' call";
+      R << ", '" << *II->getCalledFunctionName() << "' call";
     } else {
       R << ", '" << Inst.getOpcodeName() << "' instruction";
     }
diff --git a/llvm/lib/Analysis/ReplayInlineAdvisor.cpp b/llvm/lib/Analysis/ReplayInlineAdvisor.cpp
@@ -108,7 +108,7 @@ std::unique_ptr<InlineAdvice> ReplayInlineAdvisor::getAdviceImpl(CallBase &CB) {
 
   std::string CallSiteLoc =
       formatCallSiteLocation(CB.getDebugLoc(), ReplaySettings.ReplayFormat);
-  StringRef Callee = CB.getCalledFunction()->getName();
+  StringRef Callee = *CB.getCalledFunctionName();
   std::string Combined = (Callee + CallSiteLoc).str();
 
   // Replay decision, if it has one
diff --git a/llvm/lib/IR/AutoUpgrade.cpp b/llvm/lib/IR/AutoUpgrade.cpp
@@ -4398,7 +4398,7 @@ void llvm::UpgradeIntrinsicCall(CallBase *CI, Function *NewFn) {
     if (CI->getFunctionType() == NewFn->getFunctionType()) {
       // Handle generic mangling change.
       assert(
-          (CI->getCalledFunction()->getName() != NewFn->getName()) &&
+          (CI->getCalledFunctionName().value_or("") != NewFn->getName()) &&
           "Unknown function for CallBase upgrade and isn't just a name change");
       CI->setCalledFunction(NewFn);
       return;
diff --git a/llvm/lib/Target/AArch64/AArch64ISelLowering.cpp b/llvm/lib/Target/AArch64/AArch64ISelLowering.cpp
@@ -9036,12 +9036,17 @@ AArch64TargetLowering::LowerCall(CallLoweringInfo &CLI,
   auto DescribeCallsite =
       [&](OptimizationRemarkAnalysis &R) -> OptimizationRemarkAnalysis & {
     R << "call from '" << ore::NV("Caller", MF.getName()) << "' to '";
-    if (auto *ES = dyn_cast<ExternalSymbolSDNode>(CLI.Callee))
+    if (auto *ES = dyn_cast<ExternalSymbolSDNode>(CLI.Callee)) {
       R << ore::NV("Callee", ES->getSymbol());
-    else if (CLI.CB && CLI.CB->getCalledFunction())
-      R << ore::NV("Callee", CLI.CB->getCalledFunction()->getName());
-    else
+    }
+     else if (CLI.CB) {
+      const std::optional<StringRef> CalleeName =
+          CLI.CB->getCalledFunctionName();
+      R << ore::NV("Callee", CalleeName.value_or("unknown callee"));
+    }
+    else {
       R << "unknown callee";
+    }
     R << "'";
     return R;
   };
diff --git a/llvm/lib/Target/AMDGPU/AMDGPULibCalls.cpp b/llvm/lib/Target/AMDGPU/AMDGPULibCalls.cpp
@@ -1389,8 +1389,9 @@ bool AMDGPULibCalls::fold_sincos(FPMathOperator *FPOp, IRBuilder<> &B,
                        fInfo);
   const std::string PairName = PartnerInfo.mangle();
 
-  StringRef SinName = isSin ? CI->getCalledFunction()->getName() : PairName;
-  StringRef CosName = isSin ? PairName : CI->getCalledFunction()->getName();
+  const std::optional<StringRef> CalleeName = CI->getCalledFunctionName();
+  StringRef SinName = isSin ? *CalleeName : PairName;
+  StringRef CosName = isSin ? PairName : *CalleeName;
   const std::string SinCosPrivateName = SinCosLibFuncPrivate.mangle();
   const std::string SinCosGenericName = SinCosLibFuncGeneric.mangle();
 
diff --git a/llvm/lib/Target/SPIRV/SPIRVEmitIntrinsics.cpp b/llvm/lib/Target/SPIRV/SPIRVEmitIntrinsics.cpp
@@ -1638,9 +1638,9 @@ void SPIRVEmitIntrinsics::insertPtrCastOrAssignTypeInstr(Instruction *I,
     return;
 
   // collect information about formal parameter types
-  std::string DemangledName =
-      getOclOrSpirvBuiltinDemangledName(CI->getCalledFunction()->getName());
   Function *CalledF = CI->getCalledFunction();
+  std::string DemangledName =
+      getOclOrSpirvBuiltinDemangledName(CalledF->getName());
   SmallVector<Type *, 4> CalledArgTys;
   bool HaveTypes = false;
   for (unsigned OpIdx = 0; OpIdx < CalledF->arg_size(); ++OpIdx) {
diff --git a/llvm/lib/Transforms/IPO/MemProfContextDisambiguation.cpp b/llvm/lib/Transforms/IPO/MemProfContextDisambiguation.cpp
@@ -1962,7 +1962,7 @@ std::string ModuleCallsiteContextGraph::getLabel(const Function *Func,
                                                  const Instruction *Call,
                                                  unsigned CloneNo) const {
   return (Twine(Call->getFunction()->getName()) + " -> " +
-          cast<CallBase>(Call)->getCalledFunction()->getName())
+          *cast<CallBase>(Call)->getCalledFunctionName())
       .str();
 }
 
diff --git a/llvm/lib/Transforms/IPO/OpenMPOpt.cpp b/llvm/lib/Transforms/IPO/OpenMPOpt.cpp
@@ -5278,12 +5278,12 @@ struct AAFoldRuntimeCallCallSiteReturned : AAFoldRuntimeCall {
 
       CallBase *CB = dyn_cast<CallBase>(&I);
       auto Remark = [&](OptimizationRemark OR) {
+        const std::optional<StringRef> CalleeName = CB->getCalledFunctionName();
         if (auto *C = dyn_cast<ConstantInt>(*SimplifiedValue))
-          return OR << "Replacing OpenMP runtime call "
-                    << CB->getCalledFunction()->getName() << " with "
-                    << ore::NV("FoldedValue", C->getZExtValue()) << ".";
-        return OR << "Replacing OpenMP runtime call "
-                  << CB->getCalledFunction()->getName() << ".";
+          return OR << "Replacing OpenMP runtime call " << *CalleeName
+                    << " with " << ore::NV("FoldedValue", C->getZExtValue())
+                    << ".";
+        return OR << "Replacing OpenMP runtime call " << *CalleeName << ".";
       };
 
       if (CB && EnableVerboseRemarks)
diff --git a/llvm/lib/Transforms/IPO/SampleProfile.cpp b/llvm/lib/Transforms/IPO/SampleProfile.cpp
@@ -1043,8 +1043,7 @@ void SampleProfileLoader::findExternalInlineCandidate(
     // Samples may not exist for replayed function, if so
     // just add the direct GUID and move on
     if (!Samples) {
-      InlinedGUIDs.insert(
-          Function::getGUID(CB->getCalledFunction()->getName()));
+      InlinedGUIDs.insert(Function::getGUID(*CB->getCalledFunctionName()));
       return;
     }
     // Otherwise, drop the threshold to import everything that we can
diff --git a/llvm/lib/Transforms/Instrumentation/MemorySanitizer.cpp b/llvm/lib/Transforms/Instrumentation/MemorySanitizer.cpp
@@ -5458,7 +5458,7 @@ struct MemorySanitizerVisitor : public InstVisitor<MemorySanitizerVisitor> {
 
   void dumpInst(Instruction &I) {
     if (CallInst *CI = dyn_cast<CallInst>(&I)) {
-      errs() << "ZZZ call " << CI->getCalledFunction()->getName() << "\n";
+      errs() << "ZZZ call " << *CI->getCalledFunctionName() << "\n";
     } else {
       errs() << "ZZZ " << I.getOpcodeName() << "\n";
     }
diff --git a/llvm/lib/Transforms/Instrumentation/PGOMemOPSizeOpt.cpp b/llvm/lib/Transforms/Instrumentation/PGOMemOPSizeOpt.cpp
@@ -132,8 +132,8 @@ struct MemOp {
   }
   StringRef getFuncName() {
     if (auto MI = asMI())
-      return MI->getCalledFunction()->getName();
-    return asCI()->getCalledFunction()->getName();
+      return *MI->getCalledFunctionName();
+    return *asCI()->getCalledFunctionName();
   }
   bool isMemmove() {
     if (auto MI = asMI())
diff --git a/llvm/lib/Transforms/Scalar/LowerMatrixIntrinsics.cpp b/llvm/lib/Transforms/Scalar/LowerMatrixIntrinsics.cpp
@@ -2301,7 +2301,7 @@ class LowerMatrixIntrinsics {
       if (!CI->getCalledFunction())
         write("<no called fn>");
       else {
-        StringRef Name = CI->getCalledFunction()->getName();
+        StringRef Name = *CI->getCalledFunctionName();
         if (!Name.starts_with("llvm.matrix")) {
           write(Name);
           return;
diff --git a/llvm/lib/Transforms/Utils/InjectTLIMappings.cpp b/llvm/lib/Transforms/Utils/InjectTLIMappings.cpp
@@ -75,23 +75,24 @@ static void addMappingsFromTLI(const TargetLibraryInfo &TLI, CallInst &CI) {
   // bitcast (i32 (...)* @goo to i32 (i32*, ...)*)(i32* nonnull %i)`,
   // as such calls make the `isFunctionVectorizable` raise an
   // exception.
-  if (CI.isNoBuiltin() || !CI.getCalledFunction())
+  if (CI.isNoBuiltin())
     return;
 
-  StringRef ScalarName = CI.getCalledFunction()->getName();
+  const std::optional<StringRef> ScalarName = CI.getCalledFunctionName();
 
   // Nothing to be done if the TLI thinks the function is not
   // vectorizable.
-  if (!TLI.isFunctionVectorizable(ScalarName))
+  if (!ScalarName.has_value() || !TLI.isFunctionVectorizable(*ScalarName))
     return;
+
   SmallVector<std::string, 8> Mappings;
   VFABI::getVectorVariantNames(CI, Mappings);
   Module *M = CI.getModule();
   const SetVector<StringRef> OriginalSetOfMappings(Mappings.begin(),
                                                    Mappings.end());
 
   auto AddVariantDecl = [&](const ElementCount &VF, bool Predicate) {
-    const VecDesc *VD = TLI.getVectorMappingInfo(ScalarName, VF, Predicate);
+    const VecDesc *VD = TLI.getVectorMappingInfo(*ScalarName, VF, Predicate);
     if (VD && !VD->getVectorFnName().empty()) {
       std::string MangledName = VD->getVectorFunctionABIVariantString();
       if (!OriginalSetOfMappings.count(MangledName)) {
@@ -106,7 +107,7 @@ static void addMappingsFromTLI(const TargetLibraryInfo &TLI, CallInst &CI) {
 
   //  All VFs in the TLI are powers of 2.
   ElementCount WidestFixedVF, WidestScalableVF;
-  TLI.getWidestVF(ScalarName, WidestFixedVF, WidestScalableVF);
+  TLI.getWidestVF(*ScalarName, WidestFixedVF, WidestScalableVF);
 
   for (bool Predicated : {false, true}) {
     for (ElementCount VF = ElementCount::getFixed(2);
diff --git a/llvm/lib/Transforms/Utils/LibCallsShrinkWrap.cpp b/llvm/lib/Transforms/Utils/LibCallsShrinkWrap.cpp
@@ -58,7 +58,7 @@ class LibCallsShrinkWrap : public InstVisitor<LibCallsShrinkWrap> {
   bool perform() {
     bool Changed = false;
     for (auto &CI : WorkList) {
-      auto CalleeName = CI->getCalledFunctionName();
+      std::optional<StringRef> CalleeName = CI->getCalledFunctionName();
       assert(CalleeName.has_value() &&
              "perform() should apply to a non-empty callee");
 
diff --git a/llvm/lib/Transforms/Utils/SimplifyLibCalls.cpp b/llvm/lib/Transforms/Utils/SimplifyLibCalls.cpp
@@ -4041,11 +4041,11 @@ Value *LibCallSimplifier::optimizeFloatingPointLibCall(CallInst *CI,
   case LibFunc_cos:
   case LibFunc_sin:
   case LibFunc_tanh:
-    if (UnsafeFPShrink && hasFloatVersion(M, CI->getCalledFunction()->getName()))
+    if (UnsafeFPShrink && hasFloatVersion(M, *CI->getCalledFunctionName()))
       return optimizeUnaryDoubleFP(CI, Builder, TLI, true);
     return nullptr;
   case LibFunc_copysign:
-    if (hasFloatVersion(M, CI->getCalledFunction()->getName()))
+    if (hasFloatVersion(M, *CI->getCalledFunctionName()))
       return optimizeBinaryDoubleFP(CI, Builder, TLI);
     return nullptr;
   case LibFunc_fdim:
diff --git a/llvm/lib/Transforms/Vectorize/LoopVectorizationLegality.cpp b/llvm/lib/Transforms/Vectorize/LoopVectorizationLegality.cpp
@@ -759,10 +759,10 @@ bool LoopVectorizationLegality::setupOuterLoopInductions() {
 ///      {"llvm.phx.abs.i32", "", 4}
 ///    };
 static bool isTLIScalarize(const TargetLibraryInfo &TLI, const CallInst &CI) {
-  const StringRef ScalarName = CI.getCalledFunction()->getName();
+  const StringRef ScalarName = *CI.getCalledFunctionName();
   bool Scalarize = TLI.isFunctionVectorizable(ScalarName);
   // Check that all known VFs are not associated to a vector
-  // function, i.e. the vector name is emty.
+  // function, i.e. the vector name is empty.
   if (Scalarize) {
     ElementCount WidestFixedVF, WidestScalableVF;
     TLI.getWidestVF(ScalarName, WidestFixedVF, WidestScalableVF);
@@ -907,11 +907,10 @@ bool LoopVectorizationLegality::canVectorizeInstrs() {
         // If the call is a recognized math libary call, it is likely that
         // we can vectorize it given loosened floating-point constraints.
         LibFunc Func;
+        const std::optional<StringRef> FuncName = CI->getCalledFunctionName();
         bool IsMathLibCall =
-            TLI && CI->getCalledFunction() &&
-            CI->getType()->isFloatingPointTy() &&
-            TLI->getLibFunc(CI->getCalledFunction()->getName(), Func) &&
-            TLI->hasOptimizedCodeGen(Func);
+            TLI && FuncName.has_value() && CI->getType()->isFloatingPointTy() &&
+            TLI->getLibFunc(*FuncName, Func) && TLI->hasOptimizedCodeGen(Func);
 
         if (IsMathLibCall) {
           // TODO: Ideally, we should not use clang-specific language here,
diff --git a/llvm/lib/Transforms/Vectorize/VPlanRecipes.cpp b/llvm/lib/Transforms/Vectorize/VPlanRecipes.cpp
@@ -2439,7 +2439,7 @@ void VPReplicateRecipe::print(raw_ostream &O, const Twine &Indent,
   if (auto *CB = dyn_cast<CallBase>(getUnderlyingInstr())) {
     O << "call";
     printFlags(O);
-    O << "@" << CB->getCalledFunction()->getName() << "(";
+    O << "@" << *CB->getCalledFunctionName() << "(";
     interleaveComma(make_range(op_begin(), op_begin() + (getNumOperands() - 1)),
                     O, [&O, &SlotTracker](VPValue *Op) {
                       Op->printAsOperand(O, SlotTracker);
diff --git a/llvm/unittests/Analysis/InlineOrderPlugin/InlineOrderPlugin.cpp b/llvm/unittests/Analysis/InlineOrderPlugin/InlineOrderPlugin.cpp
@@ -21,7 +21,7 @@ class NoFooInlineOrder : public InlineOrder<std::pair<CallBase *, int>> {
   size_t size() override { return DefaultInlineOrder->size(); }
   void push(const std::pair<CallBase *, int> &Elt) override {
     // We ignore calles named "foo"
-    if (Elt.first->getCalledFunction()->getName() == "foo") {
+    if (*Elt.first->getCalledFunctionName() == "foo") {
       DefaultInlineOrder->push(Elt);
     }
   }
diff --git a/llvm/unittests/Analysis/PluginInlineAdvisorAnalysisTest.cpp b/llvm/unittests/Analysis/PluginInlineAdvisorAnalysisTest.cpp
@@ -35,7 +35,7 @@ class FooOnlyInlineAdvisor : public InlineAdvisor {
       : InlineAdvisor(M, FAM, IC) {}
 
   std::unique_ptr<InlineAdvice> getAdviceImpl(CallBase &CB) override {
-    if (CB.getCalledFunction()->getName() == "foo")
+    if (*CB.getCalledFunctionName() == "foo")
       return std::make_unique<InlineAdvice>(this, CB, getCallerORE(CB), true);
     return std::make_unique<InlineAdvice>(this, CB, getCallerORE(CB), false);
   }
diff --git a/llvm/unittests/Frontend/OpenMPIRBuilderTest.cpp b/llvm/unittests/Frontend/OpenMPIRBuilderTest.cpp
diff --git a/llvm/unittests/IR/IRBuilderTest.cpp b/llvm/unittests/IR/IRBuilderTest.cpp
diff --git a/llvm/unittests/Transforms/Coroutines/ExtraRematTest.cpp b/llvm/unittests/Transforms/Coroutines/ExtraRematTest.cpp

Original file line number	Diff line number	Diff line change
`@@ -153,7 +153,7 @@ void IRInstructionData::setCalleeName(bool MatchByName) {`
`153`	`153`	`}`
`154`	`154`
`155`	`155`	`if (!CI->isIndirectCall() && MatchByName)`
`156`		`- CalleeName = CI->getCalledFunction()->getName().str();`
	`156`	`+ CalleeName = CI->getCalledFunctionName()->str();`
`157`	`157`	`}`
`158`	`158`
`159`	`159`	`void IRInstructionData::setPHIPredecessors(`
Original file line number	Diff line number	Diff line change
`@@ -159,7 +159,7 @@ static void remarkFlatAddrspaceAccess(OptimizationRemarkEmitter &ORE,`
`159`	`159`	`R << "in ";`
`160`	`160`	`identifyFunction(R, Caller);`
`161`	`161`	`if (const IntrinsicInst *II = dyn_cast<IntrinsicInst>(&Inst)) {`
`162`		`- R << ", '" << II->getCalledFunction()->getName() << "' call";`
	`162`	`+ R << ", '" << *II->getCalledFunctionName() << "' call";`
`163`	`163`	`} else {`
`164`	`164`	`R << ", '" << Inst.getOpcodeName() << "' instruction";`
`165`	`165`	`}`
Original file line number	Diff line number	Diff line change
`@@ -1962,7 +1962,7 @@ std::string ModuleCallsiteContextGraph::getLabel(const Function *Func,`
`1962`	`1962`	`const Instruction *Call,`
`1963`	`1963`	`unsigned CloneNo) const {`
`1964`	`1964`	`return (Twine(Call->getFunction()->getName()) + " -> " +`
`1965`		`- cast<CallBase>(Call)->getCalledFunction()->getName())`
	`1965`	`+ *cast<CallBase>(Call)->getCalledFunctionName())`
`1966`	`1966`	`.str();`
`1967`	`1967`	`}`
`1968`	`1968`