<div dir="ltr">There are two parts of the change -- one in compiler_rt and one in LLVM.  There is an inconsistent state between.<div><br></div><div>David</div></div><div class="gmail_extra"><br><div class="gmail_quote">On Thu, May 5, 2016 at 4:03 PM, Vitaly Buka <span dir="ltr"><<a href="mailto:vitalybuka@google.com" target="_blank">vitalybuka@google.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr">This CL probably breaks this bot <a href="http://lab.llvm.org:8011/builders/sanitizer-x86_64-linux/builds/23853/steps/test%20standalone%20compiler-rt/logs/stdio" target="_blank">http://lab.llvm.org:8011/builders/sanitizer-x86_64-linux/builds/23853/steps/test%20standalone%20compiler-rt/logs/stdio</a></div><br><div class="gmail_quote"><div><div class="h5"><div dir="ltr">On Thu, May 5, 2016 at 12:47 PM David Li via llvm-commits <<a href="mailto:llvm-commits@lists.llvm.org" target="_blank">llvm-commits@lists.llvm.org</a>> wrote:<br></div></div></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div><div class="h5">This revision was automatically updated to reflect the committed changes.<br>
Closed by commit rL268667: [profile] Remove unneeded field in raw profile reader (authored by davidxl).<br>
<br>
Changed prior to commit:<br>
  <a href="http://reviews.llvm.org/D19956?vs=56240&id=56325#toc" rel="noreferrer" target="_blank">http://reviews.llvm.org/D19956?vs=56240&id=56325#toc</a><br>
<br>
Repository:<br>
  rL LLVM<br>
<br>
<a href="http://reviews.llvm.org/D19956" rel="noreferrer" target="_blank">http://reviews.llvm.org/D19956</a><br>
<br>
Files:<br>
  llvm/trunk/include/llvm/ProfileData/InstrProfReader.h<br>
  llvm/trunk/lib/ProfileData/InstrProfReader.cpp<br>
<br>
Index: llvm/trunk/include/llvm/ProfileData/InstrProfReader.h<br>
===================================================================<br>
--- llvm/trunk/include/llvm/ProfileData/InstrProfReader.h<br>
+++ llvm/trunk/include/llvm/ProfileData/InstrProfReader.h<br>
@@ -170,8 +170,9 @@<br>
   const uint64_t *CountersStart;<br>
   const char *NamesStart;<br>
   uint64_t NamesSize;<br>
+  // After value profile is all read, this pointer points to<br>
+  // the header of next profile data (if exists)<br>
   const uint8_t *ValueDataStart;<br>
-  const char *ProfileEnd;<br>
   uint32_t ValueKindLast;<br>
   uint32_t CurValueDataSize;<br>
<br>
@@ -224,6 +225,10 @@<br>
     Data++;<br>
     ValueDataStart += CurValueDataSize;<br>
   }<br>
+  const char *getNextHeaderPos() const {<br>
+      assert(atEnd());<br>
+      return (const char *)ValueDataStart;<br>
+  }<br>
<br>
   const uint64_t *getCounter(IntPtrT CounterPtr) const {<br>
     ptrdiff_t Offset = (swap(CounterPtr) - CountersDelta) / sizeof(uint64_t);<br>
Index: llvm/trunk/lib/ProfileData/InstrProfReader.cpp<br>
===================================================================<br>
--- llvm/trunk/lib/ProfileData/InstrProfReader.cpp<br>
+++ llvm/trunk/lib/ProfileData/InstrProfReader.cpp<br>
@@ -324,7 +324,6 @@<br>
   auto DataSize = swap(Header.DataSize);<br>
   auto CountersSize = swap(Header.CountersSize);<br>
   NamesSize = swap(Header.NamesSize);<br>
-  auto ValueDataSize = swap(Header.ValueDataSize);<br>
   ValueKindLast = swap(Header.ValueKindLast);<br>
<br>
   auto DataSizeInBytes = DataSize * sizeof(RawInstrProf::ProfileData<IntPtrT>);<br>
@@ -334,19 +333,17 @@<br>
   ptrdiff_t CountersOffset = DataOffset + DataSizeInBytes;<br>
   ptrdiff_t NamesOffset = CountersOffset + sizeof(uint64_t) * CountersSize;<br>
   ptrdiff_t ValueDataOffset = NamesOffset + NamesSize + PaddingSize;<br>
-  size_t ProfileSize = ValueDataOffset + ValueDataSize;<br>
<br>
   auto *Start = reinterpret_cast<const char *>(&Header);<br>
-  if (Start + ProfileSize > DataBuffer->getBufferEnd())<br>
+  if (Start + ValueDataOffset > DataBuffer->getBufferEnd())<br>
     return error(instrprof_error::bad_header);<br>
<br>
   Data = reinterpret_cast<const RawInstrProf::ProfileData<IntPtrT> *>(<br>
       Start + DataOffset);<br>
   DataEnd = Data + DataSize;<br>
   CountersStart = reinterpret_cast<const uint64_t *>(Start + CountersOffset);<br>
   NamesStart = Start + NamesOffset;<br>
   ValueDataStart = reinterpret_cast<const uint8_t *>(Start + ValueDataOffset);<br>
-  ProfileEnd = Start + ProfileSize;<br>
<br>
   std::unique_ptr<InstrProfSymtab> NewSymtab = make_unique<InstrProfSymtab>();<br>
   if (auto EC = createSymtab(*NewSymtab.get()))<br>
@@ -411,9 +408,9 @@<br>
     return success();<br>
<br>
   ErrorOr<std::unique_ptr<ValueProfData>> VDataPtrOrErr =<br>
-      ValueProfData::getValueProfData(ValueDataStart,<br>
-                                      (const unsigned char *)ProfileEnd,<br>
-                                      getDataEndianness());<br>
+      ValueProfData::getValueProfData(<br>
+          ValueDataStart, (const unsigned char *)DataBuffer->getBufferEnd(),<br>
+          getDataEndianness());<br>
<br>
   if (VDataPtrOrErr.getError())<br>
     return VDataPtrOrErr.getError();<br>
@@ -430,7 +427,8 @@<br>
 std::error_code<br>
 RawInstrProfReader<IntPtrT>::readNextRecord(InstrProfRecord &Record) {<br>
   if (atEnd())<br>
-    if (std::error_code EC = readNextHeader(ProfileEnd))<br>
+    // At this point, ValueDataStart field points to the next header.<br>
+    if (std::error_code EC = readNextHeader(getNextHeaderPos()))<br>
       return EC;<br>
<br>
   // Read name ad set it in Record.<br>
<br>
<br></div></div>
_______________________________________________<br>
llvm-commits mailing list<br>
<a href="mailto:llvm-commits@lists.llvm.org" target="_blank">llvm-commits@lists.llvm.org</a><br>
<a href="http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-commits" rel="noreferrer" target="_blank">http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-commits</a><br>
</blockquote></div>
</blockquote></div><br></div>