Fix opt-remark with hotness testcase for sample-based PGO

1. Require hotness on all remark lines with -verify.

3. Fix the samplePGO file to actually produce hotness on each line.

The second remark has hotness 60 rather 30 which I don't quite understand but
testing this is strictly better than before.  It also unblocks the commit of
D40678.

llvm-svn: 319577
This commit is contained in:
Adam Nemet 2017-12-01 19:59:42 +00:00
parent 67be3529f2
commit 2729a96bf7
2 changed files with 7 additions and 7 deletions

View File

@ -1,7 +1,7 @@
foo:0:0
0: 0
foo:29:29
0: 29
bar:29:29
9: foo:0
main:0:0
0: 0 bar:0
8: 29 foo:29
main:29:1
3: 29 bar:29

View File

@ -56,13 +56,13 @@ void bar(int x) {
// THRESHOLD-NOT: hotness
// NO_PGO: '-fdiagnostics-show-hotness' requires profile-guided optimization information
// NO_PGO: '-fdiagnostics-hotness-threshold=' requires profile-guided optimization information
// expected-remark@+1 {{foo inlined into bar with cost=always}}
// expected-remark@+1 {{foo inlined into bar with cost=always (hotness:}}
sum += foo(x, x - 2);
}
int main(int argc, const char *argv[]) {
for (int i = 0; i < 30; i++)
// expected-remark@+1 {{bar not inlined into main because it should never be inlined}}
// expected-remark@+1 {{bar not inlined into main because it should never be inlined (cost=never) (hotness:}}
bar(argc);
return sum;
}