This is the mail archive of the gcc-patches@gcc.gnu.org mailing list for the GCC project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

Re: [BUILDROBOT][PATCH] Fix mmix (unused variable)


On Fri, 18 Jul 2014, Hans-Peter Nilsson wrote:

> On Fri, 18 Jul 2014, Jan-Benedict Glaw wrote:
> > This was a build using GCC's ./contrib/config-list.mk to do the build.
> > It passes --enable-werror-always to top-level `configure', this is
> > where the -Werror comes from.
> 
> Aha.  Looks like it's of more use than theoretical pain; sounds
> like this should (effectively) be the default for *non-releases*
> when cross-compiling, with the possibility to override
> per-target.  Agreement?  Anyone against?  1/2 :)
> 
> It should be per-target because there *may* be port-specific
> constructs warned about by buggy previous-but-not-ancient
> gcc-versions, where working around the warnings would cause
> unwanted obfuscation.  (IIRC gdb does something like this.)
> 
> Is that the reason it's not the default, that there are such
> constructs in the non-port-specific parts?  But then that would
> have already been noticed through use of the config-list.mk, no?

The reason it's not the default is because the warnings are coming
from the host compiler which may be fairly old or even broken.

If we want it to make the default the we should restrict it
based on the host compiler (version).

Richard.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]