This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: RFA: Use .opt files for the ARM port
On Wed, 2005-05-04 at 19:21, Richard Sandiford wrote:
> Richard Earnshaw <rearnsha@gcc.gnu.org> writes:
> > One quick question.
> >
> > Is it not possible to merge:
> >
> > +mno-sched-prolog
> > +Target RejectNegative Report Mask(NO_SCHED_PRO)
> > +Do not move instructions into a function's prologue
> >
> > and
> >
> > +msched-prolog
> > +Target RejectNegative InverseMask(NO_SCHED_PRO) Undocumented
> >
> > into a single flag:
> >
> > msched-prolog
> > TARGET Report InverseMask(NO_SCHED_PRO)
> > Permit scheduling of a function's prologue sequence
>
> Not exactly as written, because InverseMask() doesn't declare a mask.
> It would be OK if you also had a separate:
>
> Mask(NO_SCHED_PRO)
>
> But if you're happy with advertising -msched-prolog rather than
> -mno-sched-prolog, I guess we might as well go the whole hog and call
> the target macro TARGET_SCHED_PROLOG instead. The patch below does
> that, adds MASK_SCHED_PROLOG to the default flags, and changes the three
> uses of TARGET_NO_SCHED_PRO in arm.c. There are no other changes from
> last time.
>
> Tested by building cc1 for arm-elf and checking that the flag
> still works as expected. Is the revised version OK?
This is fine.
R.