Ok 60kb is probably fine.  Lgtm<br><div class="gmail_quote"><div dir="ltr">On Mon, Aug 6, 2018 at 11:36 AM Alexandre Ganea via Phabricator <<a href="mailto:reviews@reviews.llvm.org">reviews@reviews.llvm.org</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">aganea marked 5 inline comments as done.<br>
aganea added inline comments.<br>
<br>
<br>
================<br>
Comment at: lib/DebugInfo/PDB/Native/PDBFile.cpp:473-474<br>
+  assert(StreamIndex < Layout.StreamSizes.size() && "Invalid stream index");<br>
+  if (Layout.StreamSizes[StreamIndex] == 0)<br>
+    return make_error<RawError>(raw_error_code::no_stream);<br>
   return MappedBlockStream::createIndexedStream(Layout, MsfData, StreamIndex,<br>
----------------<br>
zturner wrote:<br>
> This check looks incorrect to me.  A 0-byte stream is still a stream.<br>
Reverted this change - replaced by change in InputFile.cpp instead.<br>
<br>
<br>
================<br>
Comment at: test/tools/llvm-pdbdump/Inputs/TypeServerTest.cpp:1<br>
+// Compile with "cl /c /Zi TypeServerTest.cpp /FdTypeServerTest.pdb"<br>
+<br>
----------------<br>
zturner wrote:<br>
> Instead of checking in source code and a binary object file, can we just check in a yaml file and run it through yaml2pdb?  If you can't get pdb2yaml to produce this file (because it also crashes), then just run pdb2yaml on a good file, and delete the DBI stream.  If it's possible to get pdb2yaml to produce a PDB with a 0-byte DBI stream, that should be sufficient to test this.<br>
> <br>
> If all else fails and you really must check in binary files, then I think we only need the PDB and not the obj, and you will probably need to use `/nodefaultlib /entry:main` to keep the PDB binary small.<br>
Unfourtunately the pdb is needed, because yaml2pdb generates a default DBI stream if there's none in the yaml. Also, please note this is the pdb generated by **cl.exe**, not link.exe. The current size is 60kb.<br>
<br>
<br>
Repository:<br>
  rL LLVM<br>
<br>
<a href="https://reviews.llvm.org/D50258" rel="noreferrer" target="_blank">https://reviews.llvm.org/D50258</a><br>
<br>
<br>
<br>
</blockquote></div>