On 04 Jan 2015, at 17:42, Joerg Sonnenberger <[email protected]> wrote: > > On Sun, Jan 04, 2015 at 03:24:35PM +0000, Dimitry Andric wrote: >> However, there isn't any _LIBCPP_UNUSED define in __config yet, so that >> would have to be added, like in this proposed change. > > Shouldn't it actually be attribute *used*?
Yes, that would seem to be more logical, but clang then warns about __used__
not being applicable to class members:
In file included from test-stdexcept.cpp:1:
In file included from trunk/include/array:109:
trunk/include/stdexcept:56:39: warning: '__used__' attribute only applies to
variables and functions [-Wignored-attributes]
const char *__imp_ __attribute__((__used__));
^
trunk/include/stdexcept:56:17: warning: private field '__imp_' is not used
[-Wunused-private-field]
const char *__imp_ __attribute__((__used__));
^
2 warnings generated.
And then the -Wunused-private-field warning appears anyway. :-/
-Dimitry
signature.asc
Description: Message signed with OpenPGP using GPGMail
_______________________________________________ cfe-commits mailing list [email protected] http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits
