<div dir="ltr">Done.</div><div class="gmail_extra"><br><div class="gmail_quote">On Thu, May 5, 2016 at 10:29 AM, Vedant Kumar <span dir="ltr"><<a href="mailto:vsk@apple.com" target="_blank">vsk@apple.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">vsk accepted this revision.<br>
vsk added a comment.<br>
This revision is now accepted and ready to land.<br>
<br>
Looks good with a nit.<br>
<br>
<br>
================<br>
Comment at: lib/ProfileData/InstrProfReader.cpp:431<br>
@@ -434,1 +430,3 @@<br>
<span class="">+    // At this point, ValueDataStart field points to the next header.<br>
+    if (std::error_code EC = readNextHeader((const char *)ValueDataStart))<br>
       return EC;<br>
</span>----------------<br>
I suggest adding something like `getNextHeaderPos()` (which asserts `atEnd()`) to RawInstrProfReader for clarity.<br>
<br>
<br>
<a href="http://reviews.llvm.org/D19956" rel="noreferrer" target="_blank">http://reviews.llvm.org/D19956</a><br>
<br>
<br>
<br>
</blockquote></div><br></div>