<div dir="ltr"><div>lgtm</div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">--- a/include/clang/Basic/DiagnosticSemaKinds.td<br>

+++ b/include/clang/Basic/DiagnosticSemaKinds.td<br>@@ -2074,6 +2074,8 @@ def err_attribute_selectany_non_extern_data : Error<<br> def warn_attribute_invalid_on_definition : Warning<<br>   "'%0' attribute cannot be specified on a definition">,<br>

   InGroup<IgnoredAttributes>;<br>+def err_attribute_dll_redeclaration : Error<<br>+  "%q0 redeclared with different dll attributes">;</blockquote><div><br></div><div>Should we be more specific here?  Something like:</div>
<div>redeclaration %0 adds attribute %1 which was missing the previous declaration</div>
<div><br></div></div><div class="gmail_extra"><br><br><div class="gmail_quote">On Wed, Mar 26, 2014 at 9:12 AM, Nico Rieck <span dir="ltr"><<a href="mailto:nico.rieck@gmail.com" target="_blank">nico.rieck@gmail.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 implements semantic handling of redeclarations with dllimport<br>
or dllexport attributes. Both attributes must appear on the first<br>
declaration, and dllimport must appear on every redeclaration.<br>
<span class="HOEnZb"><font color="#888888"><br>
-Nico<br>
</font></span><br>_______________________________________________<br>
cfe-commits mailing list<br>
<a href="mailto:cfe-commits@cs.uiuc.edu">cfe-commits@cs.uiuc.edu</a><br>
<a href="http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits" target="_blank">http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits</a><br>
<br></blockquote></div><br></div>