<div dir="ltr">Sure I can do that. I'll move it to ArgTypeTraits.</div><div class="gmail_extra"><br><br><div class="gmail_quote">On Thu, Aug 21, 2014 at 4:27 PM, Aaron Ballman <span dir="ltr"><<a href="mailto:aaron@aaronballman.com" target="_blank">aaron@aaronballman.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="">On Thu, Aug 21, 2014 at 7:10 PM, Jacques Pienaar <<a href="mailto:jpienaar@google.com">jpienaar@google.com</a>> wrote:<br>

> It is to make the interface a bit easier. The dynamic matcher is at runtime<br>
> where we don't have access to the constants. So the string conversion allows<br>
> referring to attributes via a name instead of raw the enum value.<br>
<br>
</div>Ahh, okay, so this is basically a hard requirement. Ew (in the case of<br>
attributes). Those enum names are subject to change on a whim. When<br>
the enum changes, it's trivial to find breakages because the compiler<br>
will tell you immediately. With the dynamic matchers, that won't be<br>
the case. Hopefully that's a non-issue. :-)<br>
<br>
Can we implement getAttrKind as a private API within ArgTypeTraits<br>
instead of making it a public API in Basic?<br>
<span class="HOEnZb"><font color="#888888"><br>
~Aaron<br>
</font></span></blockquote></div><br><br clear="all"><div><br></div>-- <br><div dir="ltr"><div style="line-height:1.5em;padding-top:10px;margin-top:10px;color:rgb(85,85,85);font-family:sans-serif"><span style="border-width:2px 0px 0px;border-style:solid;border-color:rgb(213,15,37);padding-top:2px;margin-top:2px">Jacques Pienaar |</span><span style="border-width:2px 0px 0px;border-style:solid;border-color:rgb(51,105,232);padding-top:2px;margin-top:2px"> Platforms |</span><span style="border-width:2px 0px 0px;border-style:solid;border-color:rgb(0,153,57);padding-top:2px;margin-top:2px"> <a href="mailto:jpienaar@google.com" target="_blank">jpienaar@google.com</a> |</span><span style="border-width:2px 0px 0px;border-style:solid;border-color:rgb(238,178,17);padding-top:2px;margin-top:2px"> 765-430-6883</span></div>
</div>
</div>