<div dir="ltr">looks good</div><br><div class="gmail_quote"><div dir="ltr">On Thu, May 19, 2016 at 10:41 AM Rui Ueyama <<a href="mailto:ruiu@google.com">ruiu@google.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">ruiu updated this revision to Diff 57820.<br>
ruiu added a comment.<br>
<br>
- Update as per zturner's comment<br>
<br>
<br>
<a href="http://reviews.llvm.org/D20441" rel="noreferrer" target="_blank">http://reviews.llvm.org/D20441</a><br>
<br>
Files:<br>
  include/llvm/DebugInfo/PDB/Raw/DbiStream.h<br>
  lib/DebugInfo/PDB/Raw/DbiStream.cpp<br>
  test/DebugInfo/PDB/pdbdump-headers.test<br>
  tools/llvm-pdbdump/llvm-pdbdump.cpp<br>
<br>
Index: tools/llvm-pdbdump/llvm-pdbdump.cpp<br>
===================================================================<br>
--- tools/llvm-pdbdump/llvm-pdbdump.cpp<br>
+++ tools/llvm-pdbdump/llvm-pdbdump.cpp<br>
@@ -312,7 +312,7 @@<br>
   P.printBoolean("Has CTypes", DS.hasCTypes());<br>
   P.printBoolean("Is Stripped", DS.isStripped());<br>
   P.printObject("Machine Type", DS.getMachineType());<br>
-  P.printNumber("Number of Symbols", DS.getNumberOfSymbols());<br>
+  P.printNumber("Symbol Record Stream Index", DS.getSymRecordStreamIndex());<br>
<br>
   uint16_t Major = DS.getBuildMajorVersion();<br>
   uint16_t Minor = DS.getBuildMinorVersion();<br>
Index: test/DebugInfo/PDB/pdbdump-headers.test<br>
===================================================================<br>
--- test/DebugInfo/PDB/pdbdump-headers.test<br>
+++ test/DebugInfo/PDB/pdbdump-headers.test<br>
@@ -38,7 +38,7 @@<br>
 ; EMPTY-NEXT:   Has CTypes: No<br>
 ; EMPTY-NEXT:   Is Stripped: No<br>
 ; EMPTY-NEXT:   Machine Type: x86<br>
-; EMPTY-NEXT:   Number of Symbols: 8<br>
+; EMPTY-NEXT:   Symbol Record Stream Index: 8<br>
 ; EMPTY-NEXT:   Toolchain Version: 12.0<br>
 ; EMPTY-NEXT:   mspdb120.dll version: 12.0.31101<br>
 ; EMPTY-NEXT:   Modules [<br>
@@ -415,7 +415,7 @@<br>
 ; BIG-NEXT:   Has CTypes: No<br>
 ; BIG-NEXT:   Is Stripped: No<br>
 ; BIG-NEXT:   Machine Type: x86<br>
-; BIG-NEXT:   Number of Symbols: 9<br>
+; BIG-NEXT:   Symbol Record Stream Index: 9<br>
 ; BIG-NEXT:   Toolchain Version: 14.0<br>
 ; BIG-NEXT:   mspdb140.dll version: 14.0.23918<br>
 ; BIG-NEXT:   Modules [<br>
Index: lib/DebugInfo/PDB/Raw/DbiStream.cpp<br>
===================================================================<br>
--- lib/DebugInfo/PDB/Raw/DbiStream.cpp<br>
+++ lib/DebugInfo/PDB/Raw/DbiStream.cpp<br>
@@ -53,11 +53,11 @@<br>
   little32_t VersionSignature;<br>
   ulittle32_t VersionHeader;<br>
   ulittle32_t Age;                     // Should match InfoStream.<br>
-  ulittle16_t GSSyms;                  // Number of global symbols<br>
+  ulittle16_t GlobalSymbolStreamIndex; // Global symbol stream #<br>
   ulittle16_t BuildNumber;             // See DbiBuildNo structure.<br>
-  ulittle16_t PublicSymbolStreamIndex; // Number of public symbols<br>
+  ulittle16_t PublicSymbolStreamIndex; // Public symbols stream #<br>
   ulittle16_t PdbDllVersion;           // version of mspdbNNN.dll<br>
-  ulittle16_t SymRecords;              // Number of symbols<br>
+  ulittle16_t SymRecordStreamIndex;    // Symbol records stream #<br>
   ulittle16_t PdbDllRbld;              // rbld number of mspdbNNN.dll<br>
   little32_t ModiSubstreamSize;        // Size of module info stream<br>
   little32_t SecContrSubstreamSize;    // Size of sec. contribution stream<br>
@@ -210,7 +210,9 @@<br>
<br>
 uint32_t DbiStream::getPdbDllVersion() const { return Header->PdbDllVersion; }<br>
<br>
-uint32_t DbiStream::getNumberOfSymbols() const { return Header->SymRecords; }<br>
+uint32_t DbiStream::getSymRecordStreamIndex() const {<br>
+  return Header->SymRecordStreamIndex;<br>
+}<br>
<br>
 PDB_Machine DbiStream::getMachineType() const {<br>
   uint16_t Machine = Header->MachineType;<br>
Index: include/llvm/DebugInfo/PDB/Raw/DbiStream.h<br>
===================================================================<br>
--- include/llvm/DebugInfo/PDB/Raw/DbiStream.h<br>
+++ include/llvm/DebugInfo/PDB/Raw/DbiStream.h<br>
@@ -44,7 +44,7 @@<br>
<br>
   uint32_t getPdbDllVersion() const;<br>
<br>
-  uint32_t getNumberOfSymbols() const;<br>
+  uint32_t getSymRecordStreamIndex() const;<br>
<br>
   PDB_Machine getMachineType() const;<br>
<br>
<br>
<br>
</blockquote></div>