This is an archive of the discontinued LLVM Phabricator instance.

Differential D40663

Add a test that infinite loop has profile properly collected.
ClosedPublic

Authored by davidxl on Nov 30 2017, 10:42 AM.

Details

Reviewers
xur
vsk
davide
Commits
rG8b97870c44dd: [PGO] Add a test case for infinite loops
rCRT319463: [PGO] Add a test case for infinite loops
rL319463: [PGO] Add a test case for infinite loops
Summary

Depend on D400662

Diff Detail

Repository
rL LLVM

Event Timeline

davidxl created this revision.Nov 30 2017, 10:42 AM
Herald added a subscriber: sanjoy. · View Herald TranscriptNov 30 2017, 10:42 AM
vsk mentioned this in D40662: [PGO] Do not do counter promotion for loops without exit blocks ('infinite loops').Nov 30 2017, 11:02 AM
davide accepted this revision.Nov 30 2017, 11:17 AM
davide added a subscriber: davide.

LGTM.

This revision is now accepted and ready to land.Nov 30 2017, 11:17 AM
Closed by commit rCRT319463: [PGO] Add a test case for infinite loops (authored by davidxl). · Explain WhyNov 30 2017, 11:38 AM
Closed by commit rL319463: [PGO] Add a test case for infinite loops (authored by davidxl). · Explain Why
This revision was automatically updated to reflect the committed changes.
This revision was automatically updated to reflect the committed changes.

Revision Contents

PathSize
compiler-rt/
trunk/
test/
profile/
30 lines

Diff 124982

compiler-rt/trunk/test/profile/infinite_loop.c

// RUN: %clang_pgogen -O2 -o %t %s
// RUN: env LLVM_PROFILE_FILE=%t.profraw %run %t
// RUN: llvm-profdata show -function main -counts %t.profraw| FileCheck %s
void exit(int);
int g;
__attribute__((noinline)) void foo()
{
g++;
if (g==1000)
exit(0);
}
int main()
{
while (1) {
foo();
}
}
// CHECK: Counters:
// CHECK-NEXT: main:
// CHECK-NEXT: Hash: {{.*}}
// CHECK-NEXT: Counters: 1
// CHECK-NEXT: Block counts: [1000]