[PING^2] Waddress-of-register

["Manuel López-Ibáñez" <lopezibanez at gmail dot com>]

http://gcc.gnu.org/ml/gcc-patches/2006-12/msg01676.html

Updated, bootstrapped and regression tested again.

On 25/12/06, Manuel López-Ibáñez <lopezibanez@gmail.com> wrote:
> :ADDPATCH c++:
>
> This patch continues the effort to fix PR7651 [1].
>
> It adds a new option -Waddress-of-register that takes over the
> function of warning about subscripting an array which has been
> declared "register" or taking the address of a variable which has been
> declared "register". The new option is enabled by -Wextra, so we keep
> the current behaviour but add the ability to enable/disable this
> individual warning.
>
> Bootstrapped and regression tested with --enable-languages=all on
> i686-pc-linux-gnu.
>
> OK for mainline?
>
> [1] http://gcc.gnu.org/ml/gcc-patches/2006-12/msg01103.html
>
>
> 2006-12-25  Manuel Lopez-Ibanez  <manu@gcc.gnu.org>
>
>   PR middle-end/7651
>   * c.opt (Waddress-of-register): New.
>   * doc/invoke.texi (Waddress-of-register): Document it.
>   (Wextra): Enabled by -Wextra.
>   (C++ Language Options): Add it.
>   * c-opts.c (c_common_post_options): Enabled by -Wextra.
>
> cp/
> 2006-12-25  Manuel Lopez-Ibanez  <manu@gcc.gnu.org>
>
>   PR middle-end/7651
>   * typeck.c (build_array_ref): Replace Wextra with
>   Waddress-of-register.
>   (cxx_mark_addressable): Likewise.
>
> testsuite/
> 2006-12-25  Manuel Lopez-Ibanez  <manu@gcc.gnu.org>
>
>   PR middle-end/7651
>   * g++.dg/warn/register-var-2.C: Delete. Included in ...
>   * g++.dg/warn/Waddress-of-register.C: New.
>   * g++.dg/warn/Waddress-of-register-Wextra.C: New
>   * g++.dg/warn/Waddress-of-register-no.C: New.

Attachment: waddress-of-register.diff
Description: Text document