Re: Dealing with warnings on __attribute__((user)) when dehydra is not loaded

2009-10-18 Thread tglek
Diego Novillo wrote: Given void foo(void) __attribute__((user("bleh"))); GCC will complain foo.cc:1: warning: 'user' attribute directive ignored whenever dehydra is not loaded. Since our build system uses -Werror for every build, users are not really able to use this attribute in their dehyd

Re: Dealing with warnings on __attribute__((user)) when dehydra is not loaded

2009-10-16 Thread Tom Tromey
> "Benjamin" == Benjamin Smedberg writes: Benjamin> Dehydra is going to switch from the generic user attribute to Benjamin> specific GCC-registered attributes __attribute__((NS_final)) Benjamin> at some point when one of us can code it up. Benjamin> I don't particularly like __attribute__((p

Re: Dealing with warnings on __attribute__((user)) when dehydra is not loaded

2009-10-16 Thread Basile STARYNKEVITCH
Tom Tromey wrote: "Diego" == Diego Novillo writes: Diego> void foo(void) __attribute__((user("bleh"))); Diego> foo.cc:1: warning: 'user' attribute directive ignored Diego> We could change the compiler to never complain about the 'user' Diego> attribute, unless plugins are loaded,but that als

Re: Dealing with warnings on __attribute__((user)) when dehydra is not loaded

2009-10-16 Thread Benjamin Smedberg
On 10/16/09 12:11 PM, Tom Tromey wrote: >> "Diego" == Diego Novillo writes: > > Diego> void foo(void) __attribute__((user("bleh"))); > > Diego> foo.cc:1: warning: 'user' attribute directive ignored > > Diego> We could change the compiler to never complain about the 'user' > Diego> attribute

Re: Dealing with warnings on __attribute__((user)) when dehydra is not loaded

2009-10-16 Thread Tom Tromey
> "Diego" == Diego Novillo writes: Diego> void foo(void) __attribute__((user("bleh"))); Diego> foo.cc:1: warning: 'user' attribute directive ignored Diego> We could change the compiler to never complain about the 'user' Diego> attribute, unless plugins are loaded,but that also seems incompl

Re: Dealing with warnings on __attribute__((user)) when dehydra is not loaded

2009-10-16 Thread Basile STARYNKEVITCH
Basile STARYNKEVITCH wrote: A more sophisticated variant would be to have every attribute belonging to some attribute class, and have Wno-attributes=attributeclass but I feel that is too complex! An even more crazy idea would be to interpret the argument to -Wno-attribute as a regular expre

Re: Dealing with warnings on __attribute__((user)) when dehydra is not loaded

2009-10-16 Thread Basile STARYNKEVITCH
Diego Novillo wrote: On Fri, Oct 16, 2009 at 08:44, Basile STARYNKEVITCH wrote: I have a simpler suggestion: every attribute whose name starts with an underscore like "_user" or "_foogreen" or "_BORING" never produce any warnings, except perhaps in the plugin handling it. Yeah. Or perhaps

Re: Dealing with warnings on __attribute__((user)) when dehydra is not loaded

2009-10-16 Thread Diego Novillo
[ Posting again to gcc@ ] On Fri, Oct 16, 2009 at 09:12, Benjamin Smedberg wrote: > On 10/16/09 8:17 AM, Diego Novillo wrote: > >> void foo(void) __attribute__((user("bleh"))); >> >> GCC will complain >> >> foo.cc:1: warning: 'user' attribute directive ignored >> >> whenever dehydra is not loaded

Re: Dealing with warnings on __attribute__((user)) when dehydra is not loaded

2009-10-16 Thread Diego Novillo
On Fri, Oct 16, 2009 at 08:44, Basile STARYNKEVITCH wrote: > I am surprised & delighted that Diego mentions Dehydra. I thought it was > Taras Glek's plugin, so Mozilla (not Google) related. Dehydra still is developed by Mozilla. We are just starting to use it internally to implement some analys

Re: Dealing with warnings on __attribute__((user)) when dehydra is not loaded

2009-10-16 Thread Basile STARYNKEVITCH
Diego Novillo wrote: Given void foo(void) __attribute__((user("bleh"))); GCC will complain foo.cc:1: warning: 'user' attribute directive ignored whenever dehydra is not loaded. I am surprised & delighted that Diego mentions Dehydra. I thought it was Taras Glek's plugin, so Mozilla (not Go

Dealing with warnings on __attribute__((user)) when dehydra is not loaded

2009-10-16 Thread Diego Novillo
Given void foo(void) __attribute__((user("bleh"))); GCC will complain foo.cc:1: warning: 'user' attribute directive ignored whenever dehydra is not loaded. Since our build system uses -Werror for every build, users are not really able to use this attribute in their dehydra scripts. Has anyone