<div dir="ltr">LGTM, thanks!<div><br></div><div><br></div><div><div>+  default:</div><div>+    // Type attributes are handled elsewhere; silently move on.</div><div>+    assert(Attr.isTypeAttr() && "Non-type attribute not handled");</div>
<div>+    break;</div></div><div><br></div><div>Eventually it'd be nice to be able to TableGen a list of attributes we don't expect to see here, so we can automatically generate case labels for them, and get a compile-time error if something is missing from this switch statement.</div>
</div><div class="gmail_extra"><br><br><div class="gmail_quote">On Thu, Jan 9, 2014 at 2:28 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">Ping<br>
<div class="HOEnZb"><div class="h5"><br>
On Sat, Jan 4, 2014 at 11:33 AM, Aaron Ballman <<a href="mailto:aaron@aaronballman.com">aaron@aaronballman.com</a>> wrote:<br>
> New Years ping, with another rebased patch.  :-)<br>
><br>
> ~Aaron<br>
><br>
> On Fri, Dec 27, 2013 at 12:10 PM, Aaron Ballman <<a href="mailto:aaron@aaronballman.com">aaron@aaronballman.com</a>> wrote:<br>
>> Holiday ping, with rebased patch.<br>
>><br>
>> ~Aaron<br>
>><br>
>> On Thu, Dec 19, 2013 at 10:11 AM, Aaron Ballman <<a href="mailto:aaron@aaronballman.com">aaron@aaronballman.com</a>> wrote:<br>
>>> Ping<br>
>>><br>
>>> On Sun, Dec 15, 2013 at 8:49 AM, Aaron Ballman <<a href="mailto:aaron@aaronballman.com">aaron@aaronballman.com</a>> wrote:<br>
>>>> This patch removes the TargetAttributesSema concept and replaces it<br>
>>>> with one where the parsed attributes are responsible for knowing their<br>
>>>> target-specific nature, instead of letting Sema figure it out. This is<br>
>>>> necessary so that __has_attribute can eventually determine whether a<br>
>>>> parsed attribute applies to the given target or not.<br>
>>>><br>
>>>> All of the semantic handling which was previously in<br>
>>>> TargetAttributesSema.cpp has been moved to SemaDeclAttr.cpp, but still<br>
>>>> only applies to the specific target by asking the attribute whether it<br>
>>>> exists in the given target or not. The target-specific information is<br>
>>>> generated automatically via tablegen.<br>
>>>><br>
>>>> ~Aaron<br>
</div></div></blockquote></div><br></div>