[SPU/Build/Global] separate generror warning and gcc warning functions.

Manuel López-Ibáñez lopezibanez@gmail.com
Wed Apr 22 15:49:00 GMT 2009


I am not sure who should review this, so I am adding Andrew for the
SPU parts, Paolo for the build machinery and two global maintainers.

Updated to recent revision, bootstrapped and regression tested on
x86_64-unknown-linux-gnu with
--enable-languages=all,ada

Cheers,

Manuel.

2008/8/22 Manuel López-Ibáñez <lopezibanez@gmail.com>:
> The files errors.c and errors.h define and declare some diagnostics
> functions that are used by the gen* programs. On the other hand,
> toplev.h and diagnostic.c declare and define different diagnostics
> functions used by the rest of the compiler.
>
> We have been trying to keep the declarations of error, warning, etc.
> found in errrors.h and toplev.h compatible. However, this has no
> future at all. They do not need to be compatible since they are not
> (or should not be) used in the same places. Moreover, being compatible
> has allowed errors.h to sneak into some compiler files where it does
> not belong. (I would prefer if errors.h and errors.c would rather be
> called generrors.h and generrors.c)
>
> This version should work in SPU, although I cannot test it so testing
> in that (and any other target) is appreciated.
>
>
> Bootstrapped and regression tested on x86_64-unknown-linux-gnu with
> --enable-languages=all,ada
>
> OK for trunk?
>
> 2008-08-22  Manuel Lopez-Ibanez  <manu@gcc.gnu.org>
>
>        * Makefile.in: Update dependencies.
>        * errors.c (warning): Remove unused parameter 'opt'. Returns
>        'void'.
>        * errors.h: Remove bogus comment about compatibility.
>        (warning): Update declaration.
>        * genautomata.c: Update all calls to warning.
>        * gimple.c: Do not include errors.h. Include toplev.h.
>        * tree-ssa-structalias.c: Do not include errors.h.
>        * omega.c: Likewise.
>        * tree-ssa-reassoc.c: Likewise.
>        * config/spu/spu-c.c: Likewise.
>        * config/spu/t-spu-elf: Update dependencies.
>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: fix-generrors.diff
Type: text/x-diff
Size: 11798 bytes
Desc: not available
URL: <http://gcc.gnu.org/pipermail/gcc-patches/attachments/20090422/8861d2c0/attachment.bin>


More information about the Gcc-patches mailing list