Re: [m68k] Fix PR 14018: -malign-loops/-malign-jumps broken

[Bernardo Innocenti <bernie at develer dot com>]

Zack Weinberg wrote:
> Bernardo Innocenti <bernie@develer.com> writes:
>
> > The -malign-loops, -malign-functions and -malign-jumps are both
> > broken and superseded by the -falign-* equivalents.  Surprisingly,
> > they weren't even documented in invoke.texi.
> >
> > All considered, these options should be removed right away,
> > skipping the usual deprecation cycle.
> >
> > Regression tests on m68k-linux are still in progress.
> > OK to commit once completed with no new regressions?
>
> Given they're not documented, I say this is OK, but please wait 24
> hours for objections.

Thank you, committed.  I must confess I've not been
able to run regression tests because both mainline
and 3.4-branch are currently broken on m68k for
other reasons.

I'll follow up shortly with a description of the
3.4-branch problem.


> > 2004-05-29  Bernardo Innocenti  <bernie@develer.com>
> >
> > 	PR target/14018
> > 	* config/m68k/m68k.c (m68k_align_loops_string, m68k_align_jumps_string,
> > 	m68k_align_funcs_string, m68k_align_loops, m68k_align_jumps,
> > 	m68k_align_funcs): Remove.
> > 	(override_options): Remove code to handle -malign-* options.
> > 	* config/m68k/m68k.h (TARGET_OPTIONS): Remove -malign-* options.
> > 	(FUNCTION_BOUNDARY, LOOP_ALIGN, LOOP_ALIGN_AFTER_BARRIER): Remove.
> > 	(m68k_align_loops_string, m68k_align_jumps_string,
> > 	m68k_align_funcs_string, m68k_align_loops, m68k_align_jumps,
> > 	m68k_align_funcs): Remove definitions.


--
 // Bernardo Innocenti - Develer S.r.l., R&D dept.
\X/  http://www.develer.com/