<div dir="ltr">LGTM.</div><div class="gmail_extra"><br><div class="gmail_quote">On Mon, Sep 8, 2014 at 2:20 AM, Dan Liew <span dir="ltr"><<a href="mailto:dan@su-root.co.uk" target="_blank">dan@su-root.co.uk</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hi Sean,<br>
<br>
This patch fixes a mistake in the insertvalue instruction example.<br>
Trying to assemble this with llvm-as<br>
<br>
```<br>
define void @main(float %val)<br>
{<br>
    %agg1 = insertvalue {i32, float} undef, i32 1, 0              ;<br>
yields {i32 1, float undef}<br>
    %agg2 = insertvalue {i32, float} %agg1, float %val, 1         ;<br>
yields {i32 1, float %val}<br>
    %agg3 = insertvalue {i32, {float}} %agg1, float %val, 1, 0    ;<br>
yields {i32 undef, float %val}<br>
    ret void<br>
}<br>
```<br>
<br>
gives the following error<br>
<br>
```<br>
llvm-as: agg.ll:5:40: error: '%agg1' defined with type '{ i32, float }'<br>
    %agg3 = insertvalue {i32, {float}} %agg1, float %val, 1, 0    ;<br>
yields {i32 undef, float %val}<br>
```<br>
<br>
Which isn't surprising because {i32, {float}} and {i32, float} are not<br>
the same type.<br>
<br>
Using my modified version assembles without error so I think what I've<br>
done is correct.<br>
<br>
Is this okay to commit?<br>
<br>
Thanks,<br>
Dan.<br>
</blockquote></div><br></div>