On Thu, Jun 27, 2013 at 1:06 PM, Nico Weber <[email protected]> wrote:
>> > On Wed, Jun 12, 2013 at 7:02 PM, Richard Smith 
>> > <[email protected]> wrote:
>> >> Add -Wdeprecated warnings and fixits for things deprecated in C++11:
>> >>  - 'register' storage class
>
>> >> +def warn_deprecated_register : Warning<
>> >> +  "'register' storage class specifier is deprecated">, 
>> >> InGroup<Deprecated>;
>
> Is this a useful warning? Code generated by flex for example contains
> register variables, and there doesn't seem much harm in that. Should this be
> in -pedantic?

IMHO, -Wdeprecated is the exactly right place for warnings about
deprecated language constructs. I do think the warning ought to be
individually toggleable under -Wdeprecated-register as well as being
part of the <Deprecated> group; I had actually thought that that was
Richard's plan. It wasn't?

–Arthur

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

Reply via email to