[cfe-commits] r158317 - in /cfe/trunk: include/clang/Basic/DiagnosticGroups.td test/FixIt/fixit.cpp

[Jeff Walden]

On 06/11/2012 01:22 PM, Chandler Carruth wrote:
> __attribute__((unused)) should work?

That doesn't work.  I tried the trick on SpiderMonkey today in a case where we were using a private field to pad a class to a particular size, and I got this:

> ../vm/String.h:588:20: warning: 'unused' attribute only applies to variables, functions and labels

So __attribute__((unused)) definitely doesn't work on fields right now.  It probably should.  Whether that's the right fix for this particular case where the warning's not useful, I dunno.

Jeff