[PATCH] D16258: [PGO] Profile Summary Index profile writer and reader support

[Easwaran Raman via llvm-commits]

eraman added inline comments.

================
Comment at: include/llvm/ProfileData/InstrProf.h:743
@@ +742,3 @@
+  /// Max internal block count of the program (excluding entry blocks).
+  uint64_t MaxInternalBlockCount;
+  /// Total number of instrumented blocks/edges.
----------------
This is not very useful. In practice we should expect MaxInternalBlockCount == MaxBlockCount

================
Comment at: include/llvm/ProfileData/InstrProfReader.h:345
@@ -344,1 +344,3 @@
 
+  // Read the profile summary. Return a pointer pointing to one byte pass the
+  // end of the summary data if it exists or the input \c Cur.
----------------
Nit: pass -> past


http://reviews.llvm.org/D16258