On Sat, Mar 10, 2012 at 5:25 AM, Sandra Loosemore <san...@codesourcery.com> wrote: > When I was working on cleanups to invoke.texi I noticed this bit in the > SH Options section: > > @item -madjust-unroll > @opindex madjust-unroll > Throttle unrolling to avoid thrashing target registers. > This option only has an effect if the GCC code base supports the > TARGET_ADJUST_UNROLL_MAX target hook. > > Well, the current manual is supposed to document what the current GCC > code base does, so I looked up this target hook and found it was deleted > in 2007 with a ChangeLog note that it has been dead since gcc 4.0.0. > However the command-line option is still present. :-S > > Any reason why we should not delete this entirely, per the attached patch? > I haven't built/tested this (except for the manual) but can do that if the > patch isn't sufficiently obvious.
Command-line options have to be preserved for backward-compatibility so old makefiles do not fail. Though the entry in the .opt file should match madjust-unroll Target Ignore Does nothing. Preserved for backward compatibility. The documentation can be removed, of course. Richard. > -Sandra > > > 2012-03-09 Sandra Loosemore <san...@codesourcery.com> > > gcc/ > * config/sh/sh.opt (madjust-unroll): Delete. > * config/sh/sh.h (TARGET_OPT_DEFAULT): Don't use MASK_ADJUST_UNROLL. > * doc/invoke.texi (Option Summary): Remove -madjust-unroll. > (SH Options): Likewise.