<div class="gmail_quote">On Mon, Apr 16, 2012 at 3:21 PM, Duncan Sands <span dir="ltr"><<a href="mailto:baldrick@free.fr">baldrick@free.fr</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<div id=":67b">Add convenience methods to MDBuilder for attaching metadata to instructions,<br>
and retrieving it from instructions.  I don't have a use for this but is seems<br>
logical for it to exist.  While there, remove some 'const' markings from methods<br>
which are in fact 'const' in practice, but aren't logically 'const'.<br></div></blockquote></div><br><div>I'm not really a fan of this interface.</div><div><br></div><div>If we want these convenience methods, they should be in Instruction, not on MDBuilder. Fundamentally, MDBuilder should be a MD-producing entity, not an IR-mutating entity.</div>