<div dir="ltr">I'll wait a bit since I'm in the middle of a separate large patch and in no hurry to commit this, and I don't want to switch branches :)</div><br><div class="gmail_quote"><div dir="ltr">On Tue, Jun 13, 2017 at 9:45 AM Reid Kleckner 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">rnk added a subscriber: inglorion.<br>
rnk added a comment.<br>
<br>
I skimmed over the rest of the code, and I didn't have any major comments. Do you want to give @inglorion a chance to review, or go ahead and commit?<br>
<br>
<br>
<br>
================<br>
Comment at: lld/test/COFF/Inputs/pdb1.yaml:33-34<br>
+              Version:         'Microsoft (R) Optimizing Compiler'<br>
+      - !Symbols<br>
+        Records:<br>
+          - Kind:            S_GPROC32_ID<br>
----------------<br>
zturner wrote:<br>
> rnk wrote:<br>
> > Is there a way we can fold !Symbols and Records: together? They're both plural, but !Symbols always consists of a list of records.<br>
> Looked into this and while I think it's technically supported by the YAML spec (What goes under a tag is defined to be "application specific", so I guess it could be anything, and [[ <a href="http://www.yaml.org/spec/1.2/spec.html#id2761292" rel="noreferrer" target="_blank">http://www.yaml.org/spec/1.2/spec.html#id2761292</a> | example 2.24 ]] even seems to be an exact match for what we want to do (e.g. the `!shape` tag followed immediately by a list)), I don't think our YAML parser supports it.  In order to map a block sequence, we have to give it a name.  The specification for [[ <a href="http://www.yaml.org/spec/1.2/spec.html#style/block/sequence" rel="noreferrer" target="_blank">http://www.yaml.org/spec/1.2/spec.html#style/block/sequence</a> | block sequences ]] doesn't seem to require that a block sequence have a name, but otoh the syntax used in the yaml spec is hard to make sense of, so maybe I missed something.<br>
><br>
> TL;DR - Probably could do it, but non-trivial.<br>
Oh well.<br>
<br>
<br>
<a href="https://reviews.llvm.org/D34127" rel="noreferrer" target="_blank">https://reviews.llvm.org/D34127</a><br>
<br>
<br>
<br>
</blockquote></div>