<div dir="ltr">Thanks, I committed this patch with the added tests for the non-standard alignment in r242707.</div><div class="gmail_extra"><br><div class="gmail_quote">2015-07-17 11:02 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-Jul-17, at 10:18, Alex L <<a href="mailto:arphaman@gmail.com">arphaman@gmail.com</a>> wrote:<br>
><br>
> Hello,<br>
><br>
> This patch depends on the patch D10280 (AsmParser: Add a function to parse a standalone type and value).<br>
><br>
> This patch implements initial serialization of the machine constant pool and the constant pool<br>
> index machine operand. The constant pool is serialized by using a YAML sequence of YAML<br>
> mappings that represent the constant values, e.g:<br>
><br>
>   constants:<br>
>     - id:          0<br>
>       value:       'double 3.250000e+00'<br>
>       alignment:   8<br>
>     - id:          1<br>
>       value:       'float 6.250000e+00'<br>
>       alignment:   4<br>
><br>
> Target-specific constant pool values aren't serialized by this patch.<br>
><br>
> The constant pool index machine operands use the following syntax:<br>
><br>
>   %const.<id><br>
><br>
> Thanks,<br>
> Alex<br>
</div></div>> <0001-MIR-Serialization-Initial-serialization-of-constant-.patch><br>
<br>
Please add a test that checks that non-standard alignments are<br>
respected -- that inferred alignments don't override specified ones.<br>
<br>
With that, LGTM.</blockquote></div><br></div>