Do not read callee name when matching IR to profile as it is not used.

[danielcdh]

Summary

Callee name is not used to identify a callsite now, so do not read it during annotation.

[dnovillo]

LGTM. Thanks.

I was going to ask about a test case, but we already have test cases that cover this, right?

[danielcdh]

Yep, we do have testcase that covers the annotation code.

Thanks,
Dehao

[dnovillo]

Cool, thanks.

lib/Transforms/IPO/SampleProfile.cpp

 /// \returns the FunctionSamples pointer to the inlined instance.
 const FunctionSamples *
 SampleProfileLoader::findFunctionSamples(const Instruction &Inst) const {
   SmallVector<LineLocation, 10> S;
   const DILocation *DIL = Inst.getDebugLoc();
   if (!DIL) {
     return Samples;
   }
-  StringRef CalleeName;
-  for (const DILocation *DIL = Inst.getDebugLoc(); DIL;
-       DIL = DIL->getInlinedAt()) {
+  for (DIL = DIL->getInlinedAt(); DIL; DIL = DIL->getInlinedAt()) {
     DISubprogram *SP = DIL->getScope()->getSubprogram();
     if (!SP)
       return nullptr;
-    if (!CalleeName.empty()) {
-      S.push_back(LineLocation(getOffset(DIL->getLine(), SP->getLine()),
-                               DIL->getDiscriminator()));
-    }
-    CalleeName = SP->getLinkageName();
-  }
   if (S.size() == 0)
     return Samples;
   const FunctionSamples *FS = Samples;
   for (int i = S.size() - 1; i >= 0 && FS != nullptr; i--) {
     FS = FS->findFunctionSamplesAt(S[i]);
   }
   return FS;
 }