r183881 - Add -Wdeprecated warnings and fixits for things deprecated in C++11:

[Arthur O'Dwyer]

On Thu, Jun 27, 2013 at 1:06 PM, Nico Weber <thakis at chromium.org> wrote:
>> > On Wed, Jun 12, 2013 at 7:02 PM, Richard Smith <richard-llvm at metafoo.co.uk> 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