This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [PATCH] (Partial) Implementation of simplificaiton of CSHIFT
- From: Steve Kargl <sgk at troutmask dot apl dot washington dot edu>
- To: Eric Botcazou <ebotcazou at adacore dot com>
- Cc: gcc-patches at gcc dot gnu dot org, "H.J. Lu" <hjl dot tools at gmail dot com>, Paul Richard Thomas <paul dot richard dot thomas at gmail dot com>, "fortran at gcc dot gnu dot org" <fortran at gcc dot gnu dot org>
- Date: Sat, 21 Nov 2015 14:39:23 -0800
- Subject: Re: [PATCH] (Partial) Implementation of simplificaiton of CSHIFT
- Authentication-results: sourceware.org; auth=none
- References: <20151120005836 dot GA53763 at troutmask dot apl dot washington dot edu> <20151121192622 dot GA81197 at troutmask dot apl dot washington dot edu> <20151121200735 dot GA55464 at troutmask dot apl dot washington dot edu> <6036192 dot xTO5p6c5QT at polaris>
On Sat, Nov 21, 2015 at 11:26:17PM +0100, Eric Botcazou wrote:
> > + * simplify.c (gfc_simplify_cshift): Work around bootstrap issues
> > + due to inappropriate warning options.
>
> The warning options are appropriate, any dead code can potentially hide a bug
> and should be flagged; every static analyzer will also do it and we would soon
> have PRs opened with bugzilla if we tolerated it.
>
I disgree.
If bootstrap is going to enforce -Werror -Wunused-*, then
this should be the default for any possible invocation of
make.
--
Steve