r203593 - PGO: Clarify the testing README

Duncan P. N. Exon Smith dexonsmith at apple.com
Tue Mar 11 11:18:12 PDT 2014


Author: dexonsmith
Date: Tue Mar 11 13:18:12 2014
New Revision: 203593

URL: http://llvm.org/viewvc/llvm-project?rev=203593&view=rev
Log:
PGO: Clarify the testing README

<rdar://problem/16276448>

Modified:
    cfe/trunk/test/Profile/README

Modified: cfe/trunk/test/Profile/README
URL: http://llvm.org/viewvc/llvm-project/cfe/trunk/test/Profile/README?rev=203593&r1=203592&r2=203593&view=diff
==============================================================================
--- cfe/trunk/test/Profile/README (original)
+++ cfe/trunk/test/Profile/README Tue Mar 11 13:18:12 2014
@@ -1,8 +1,16 @@
-These are tests for instrumentation based profiling. This specifically
-means the -fprofile-instr-generate and -fprofile-instr-use driver flags.
+These are tests for instrumentation based profiling.  This specifically means
+the -fprofile-instr-generate and -fprofile-instr-use driver flags.
 
-Tests in this directory should test both the generation of profile
-data, and the use of that same data. As such, the input source file
-should include a main function such that building with -use and
-running the resulting program would generate the input data that is
-consumed in the -generate case.
+Tests in this directory should usually test both:
+
+  - the generation of instrumentation (-fprofile-instr-generate), and
+  - the use of profile data from instrumented runs (-fprofile-instr-use).
+
+In order to test -fprofile-instr-use without actually running an instrumented
+program, .profdata files are checked into Inputs/.
+
+The input source files must include a main function such that building with
+-fprofile-instr-generate and running the resulting program generates the same
+.profdata file that is consumed by the tests for -fprofile-instr-use.  Even
+tests that only check -fprofile-instr-use should include such a main function,
+so that profile data can be regenerated as the .profdata file format evolves.





More information about the cfe-commits mailing list