[PATCH] D132094: [profile] Create only prof header when no counters

[Gulfem Savrun Yeniceri via Phabricator via llvm-commits]

gulfem marked 5 inline comments as done.
gulfem added inline comments.


================
Comment at: compiler-rt/test/profile/Posix/instrprof-empty-profile.c:20
+// RAW-PROFILE-HEADER-ONLY: Instrumentation level: Front-end
+// RAW-PROFILE-HEADER-ONLY-NEXT: Total functions: 0
+// RAW-PROFILE-HEADER-ONLY-NEXT: Maximum function count: 0
----------------
ellis wrote:
> I think this might be the only important line to check. Do we need to check the other lines too?
Appreciate for all the feedback @ellis! 
I prefer to check all the other lines to make sure that we have only 0s.
For this test it might not be important, but for the shared library test (`instrprof-shared-empty-profile.test`), I would like to verify the accurate values. 


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D132094/new/

https://reviews.llvm.org/D132094