<div dir="ltr">Looks great! Just a couple of questions:<div><br></div><div>1) In CalculateDiagnostic, it looks like you sometimes pick a diagnostic that's not an exact match for the subjects list. What cases do we hit where we don't have an exact match, and have you considered extending the list to cover those too? (I also wonder whether we can do better here -- maybe by including Attr.td into DiagnosticSemaKinds.td and generating extra diagnostics from Attr.td or similar -- but I don't want that blocking this patch!).</div>
<div><br></div><div>2) In GenerateAppertainsTo, you bail out for an attribute with custom parsing. I find that surprising -- that flag affects how the attribute is parsed into an AttributeList, but I don't see why it should have any implications on subject checking. Checking SemaHandler here would make sense to me, but that looks unnecessary. What's this for?</div>
</div><div class="gmail_extra"><br><br><div class="gmail_quote">On Tue, Nov 26, 2013 at 12:51 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">This patch lays the basic groundwork for table generating the<br>
diagnostics for attribute subjects. It does this by making some<br>
modifications to the way subjects are listed in Attr.td, and updating<br>
the attr emitter to handle the new constructs.<br>
<br>
I have disabled some attribute subject lines on purpose in Attr.td;<br>
this part is a WIP with the goal being to restore those subjects<br>
incrementally. By commenting them out, it leaves the original behavior<br>
the same as before for those attributes and so those are not<br>
functionality changes.<br>
<span class="HOEnZb"><font color="#888888"><br>
~Aaron<br>
</font></span></blockquote></div><br></div>