[CSSPGO] Fix a typo in SampleContextTracker · llvm/llvm-project@ab5ac65 (original) (raw)

Skip to content

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Sign up

Commit ab5ac65

[CSSPGO] Fix a typo in SampleContextTracker

Fixing a typo in SampleContextTracker to use debug name when debug linkage name is no present. This should only affect C programs. Saw 0.6% perf win on Cinder which is mostly C code. Reviewed By: wenlei, wmi Differential Revision: https://reviews.llvm.org/D106599

File tree

1 file changed

lines changed

1 file changed

lines changed

Original file line number Diff line number Diff line change
@@ -433,8 +433,7 @@ ContextTrieNode *SampleContextTracker::getContextFor(const DILocation *DIL) {
433 433 if (Name.empty())
434 434 Name = PrevDIL->getScope()->getSubprogram()->getName();
435 435 S.push_back(
436 -std::make_pair(FunctionSamples::getCallSiteIdentifier(DIL),
437 - PrevDIL->getScope()->getSubprogram()->getLinkageName()));
436 +std::make_pair(FunctionSamples::getCallSiteIdentifier(DIL), Name));
438 437 PrevDIL = DIL;
439 438 }
440 439