<pre style="word-wrap:break-word;white-space:pre-wrap">+def Mips16 : InheritableAttr {
+  let Spellings = [GNU<"mips16">];
+  let Subjects = [Function];
</pre><div>Attributes which g++-4.8 accepts should also have a CXX11<"gnu", "mips16"> spelling.</div><br><div class="gmail_quote">On Mon, Jan 14, 2013 at 2:31 PM, Dmitri Gribenko <span dir="ltr"><<a href="mailto:gribozavr@gmail.com" target="_blank">gribozavr@gmail.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="im">On Mon, Jan 14, 2013 at 8:42 AM, Reed Kotler <<a href="mailto:rkotler@mips.com">rkotler@mips.com</a>> wrote:<br>

> Forget to say that I need to have this patch reviewd.<br>
<br>
</div>+__attribute((nomips16)) int a; // expected-warning {{attribute only<br>
applies to functions}}<br>
<br>
I am concerned by the fact that this is just a warning -- I would<br>
suggest to make it an error.  This is a new attribute, so there's no<br>
code abusing this attribute (we don't need to be compatible with it).<br>
<br>
What does gcc do?<br>
<br>
+void __attribute__((nomips16)) foo32(); // expected-warning {{unknown<br>
attribute 'nomips16' ignored}}<br>
<br>
Same here.  What does gcc do in this case?  (I know that this behavior<br>
can not be changed without some refactoring.  But if GCC gives an<br>
error, we should, too.  So we would leave a comment in the test, or<br>
better -- do the required refactoring...)<br>
<span class="HOEnZb"><font color="#888888"><br>
Dmitri<br>
<br>
--<br>
main(i,j){for(i=2;;i++){for(j=2;j<i;j++){if(!(i%j)){j=0;break;}}if<br>
(j){printf("%d\n",i);}}} /*Dmitri Gribenko <<a href="mailto:gribozavr@gmail.com">gribozavr@gmail.com</a>>*/<br>
</font></span><div class="HOEnZb"><div class="h5">_______________________________________________<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>
</div></div></blockquote></div><br>