This is the mail archive of the
binutils@sources.redhat.com
mailing list for the binutils project.
Re: PATCH: Turn mips relaxation if appropriate (Re: MIPS gas relaxation still doesn't work)
On Mon, Oct 14, 2002 at 04:58:00PM +0100, Richard Sandiford wrote:
> "H. J. Lu" <hjl@lucon.org> writes:
> > Since no other assemblers warn for relaxation by default, the mips GNU
> > assembler shouldn't be different. That means branch relaxation is just
> > like another form of macro. However, if gcc wishes to check its own
> > bug, it can pass --relax to gas. But it is not required to get branch
> > relaxation.
>
> But your patch effectively changes --relax-branch to mean
> "warn about branch relaxations that are otherwise applied
> by default if -mno-embedded-pic". At the same time it's
> still acting as "enable branch relaxation" when used with
> -membedded-pic. Seems a bit convoluted.
Yes, the --relax-branch/--no-relax-branch options should be
removed/changed.
>
> At the end of the day, it is only a warning. If you really
> want a way of turning it off, why not put it under the control
> of a separate flag?
The warning should be off by default if branch relaxation is on by
default. It may be useful to have a separate flag for gcc to check
its own bug. But gas shouldn't do it on its own.
H.J.