Oct 23, 2010 kl. 9:11 PM skrev John McCall:

> On Oct 22, 2010, at 4:37 PM, Anders Carlsson wrote:
>> Author: andersca
>> Date: Fri Oct 22 18:37:08 2010
>> New Revision: 117184
>> 
>> URL: http://llvm.org/viewvc/llvm-project?rev=117184&view=rev
>> Log:
>> Warn if a variable marked with the "unused" attribute is used. Patch by 
>> Darin Adler!
> 
> Er.  Traditionally this attribute is put on declarations to white-list them 
> for -Wunused;  i.e. to say that it's okay if they aren't used, not to say 
> that it's not okay if they are used.  This breaks a *lot* of expected, 
> perfectly idiomatic uses of __attribute__((unused)).  Furthermore, AFAICT 
> __attribute__((unavailable)) already captures the desired semantics, at least 
> for (ahem) projects that always build with -Werror.
> 
> Please revert this or at least take it out of -Wunused.

Done in r117219.

- Anders

_______________________________________________
cfe-commits mailing list
[email protected]
http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits

Reply via email to