<div dir="ltr">Thanks, this got committed in r <span style="font-size:13px">245740.</span></div><div class="gmail_extra"><br><div class="gmail_quote">2015-08-21 12:28 GMT-07:00 Duncan P. N. Exon Smith <span dir="ltr"><<a href="mailto:dexonsmith@apple.com" target="_blank">dexonsmith@apple.com</a>></span>:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="HOEnZb"><div class="h5"><br>
> On 2015-Aug-20, at 17:14, Alex L <<a href="mailto:arphaman@gmail.com">arphaman@gmail.com</a>> wrote:<br>
><br>
> Hi,<br>
><br>
> This patch extends the 'SlotMapping' and includes mappings for named and<br>
> numbered types in it. The LLParser is extended accordingly to fill out those<br>
> mappings at the end of module parsing.<br>
><br>
> This information is useful when we want to parse standalone constant values<br>
> at a later stage using the 'parseConstantValue' method. The constant values<br>
> can be constant expressions, which can contain references to types. In order<br>
> to parse such constant values, we have to restore the internal named and<br>
> numbered mappings for the types in LLParser, otherwise the parser will report<br>
> a parsing error. Therefore, this patch also introduces a new method called<br>
> 'restoreParsingState' to LLParser, which uses the slot mappings to restore<br>
> some of its internal parsing state.<br>
><br>
> This patch is required to serialize constant value pointers in the machine<br>
> memory operands for the MIR format.<br>
><br>
> Cheers,<br>
> Alex<br>
</div></div>> <0001-AsmParser-Save-and-restore-the-parsing-state-of-type.patch><br>
<br>
LGTM!</blockquote></div><br></div>