Skip to content

Commit 175288a

Browse files
committed
Add sample-profile-suffix-elision-policy attribute with -funique-internal-linkage-names.
Adding sample-profile-suffix-elision-policy attribute to functions whose linkage names are uniquefied so that their unique name suffix won't be trimmed when applying AutoFDO profiles. Reviewed By: dblaikie Differential Revision: https://reviews.llvm.org/D94455
1 parent 6166b91 commit 175288a

File tree

2 files changed

+3
-1
lines changed

2 files changed

+3
-1
lines changed

llvm/lib/Transforms/Utils/UniqueInternalLinkageNames.cpp

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -43,6 +43,7 @@ static bool uniqueifyInternalLinkageNames(Module &M) {
4343
for (auto &F : M) {
4444
if (F.hasInternalLinkage()) {
4545
F.setName(F.getName() + ModuleNameHash);
46+
F.addFnAttr("sample-profile-suffix-elision-policy", "selected");
4647
// Replace linkage names in the debug metadata.
4748
if (DISubprogram *SP = F.getSubprogram()) {
4849
if (SP->getRawLinkageName()) {

llvm/test/Transforms/UniqueInternalLinkageNames/unique-internal-linkage-names.ll

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -42,8 +42,9 @@ entry:
4242
; O2: Running pass: UniqueInternalLinkageNamesPass
4343
; O2: Running pass: SampleProfileProbePass
4444

45-
; UNIQUE: define internal i32 @foo.__uniq.{{[0-9]+}}()
45+
; UNIQUE: define internal i32 @foo.__uniq.{{[0-9]+}}() [[ATTR:#[0-9]+]]
4646
; UNIQUE: ret {{.*}} @foo.__uniq.{{[0-9]+}} {{.*}}
47+
; UNIQUE: attributes [[ATTR]] = {{{.*}} "sample-profile-suffix-elision-policy"="selected" {{.*}}}
4748

4849
; DBG: distinct !DISubprogram(name: "foo", linkageName: "foo.__uniq.{{[0-9]+}}", scope: ![[#]]
4950
; DBG: !DISubprogram(name: "foo", linkageName: "foo.__uniq.{{[0-9]+}}", scope: ![[#]]

0 commit comments

Comments
 (0)