Hi, Richard Guenther <richard.guent...@gmail.com> wrote: > 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. ,A (B:-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.
The patch is OK with the change suggested by Richard. Thanks! Regards, kaz