Hi Duncan,<br><br>Looking through the LangRef as requested :)<div><br></div><div><div>+    A metadata node is a structure-like constant without a type (when</div><div>+    referenced by ``call`` instructions, they use :ref:`metadata type</div><div>+    <t_metadata>`). For example: "``!{ value i32 0, !"test" }``". Unlike other</div><div>+    constants that are meant to be interpreted as part of the instruction</div><div>+    stream, metadata is a place to attach additional information such as debug</div><div>+    info.</div></div><div><br></div><div>This is fairly opaque wording. Honestly I'd prefer "a constant tuple without types". The last probably needs/wants rewording as well, it's just not fabulous. Perhaps even an actual separate section would work well for metadata now that it's separate from the value hierarchy?</div><div><br></div><div>The rest of the changes look fairly mechanical given the new description of the format.</div><div><br></div><div>Thanks!</div><div><br></div><div>-eric</div><div><br></div><br><div class="gmail_quote">On Wed Dec 10 2014 at 10:42:34 PM Duncan P. N. Exon Smith <<a href="mailto:dexonsmith@apple.com">dexonsmith@apple.com</a>> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><br>
> On 2014 Dec 10, at 20:35, Robinson, Paul <<a href="mailto:Paul_Robinson@playstation.sony.com" target="_blank">Paul_Robinson@playstation.<u></u>sony.com</a>> wrote:<br>
><br>
> Note: in case it's confusing, some of the testcase changes in the code<br>
> patch are killing newlines so the upgrade script can do it's magic.<br>
><br>
> That part really feels like a purely mechanical change that oughta be separate.<br>
> --paulr<br>
<br>
Thanks Paul, good point.  Committed that in r224002.<br>
<br>
Attaching the rebased patch.  Let me know if you have any comments on LangRef!<br>
<br>
<br>
><br>
> From: <a href="mailto:llvm-commits-bounces@cs.uiuc.edu" target="_blank">llvm-commits-bounces@cs.uiuc.<u></u>edu</a> [mailto:<a href="mailto:llvm-commits-bounces@cs.uiuc.edu" target="_blank">llvm-commits-bounces@<u></u>cs.uiuc.edu</a>] On Behalf Of Duncan P. N. Exon Smith<br>
> Sent: Wednesday, December 10, 2014 6:17 PM<br>
> To: llvm-commits<br>
> Subject: [PATCH] IR: Make metadata typeless in assembly<br>
><br>
> Now that `Metadata` is typeless, reflect that in the assembly.<br>
><br>
> There's really only one reasonably thing to do here: strip the<br>
> `metadata` type (except in arguments to `call` instructions, where it<br>
> still has meaning), and add the symmetric `value` keyword for all<br>
> metadata operands that bridge to `Value`.<br>
><br>
> I'm confident in the code changes (although obviously I'll relish any<br>
> criticism offered), but I'd love a review of `docs/LangRef.rst`.  I<br>
> wasn't really sure how much exposition was appropriate, so I kept it<br>
> pretty simple.<br>
><br>
> Also, I wrote an upgrade script that handle all of the tests in llvm<br>
> after applying the main patch.  It even handles much of cfe (doing<br>
> mostly the right thing with `CHECK` lines).  I'll be attaching it the PR<br>
> after I make my commit, but feel free to spot-check its enormous result,<br>
> which I've attached as llvm-tests.patch.  (I'll squash that in with the<br>
> code changes when I commit.)<br>
><br>
> Note: in case it's confusing, some of the testcase changes in the code<br>
> patch are killing newlines so the upgrade script can do it's magic.<br>
><br>
> ______________________________<u></u>_________________<br>
> llvm-commits mailing list<br>
> <a href="mailto:llvm-commits@cs.uiuc.edu" target="_blank">llvm-commits@cs.uiuc.edu</a><br>
> <a href="http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits" target="_blank">http://lists.cs.uiuc.edu/<u></u>mailman/listinfo/llvm-commits</a><br>
<br>
______________________________<u></u>_________________<br>
llvm-commits mailing list<br>
<a href="mailto:llvm-commits@cs.uiuc.edu" target="_blank">llvm-commits@cs.uiuc.edu</a><br>
<a href="http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits" target="_blank">http://lists.cs.uiuc.edu/<u></u>mailman/listinfo/llvm-commits</a><br>
</blockquote></div>