On Wed, 17 Jan 2024, Jan Hubicka wrote:
> > On Wed, 17 Jan 2024, Jan Hubicka wrote:
> >
> > > >
> > > > I meant the new option might be named -fmin-function-alignment=
> > > > rather than -falign-all-functions because of how it should
> > > > override all other options.
> > >
> > > I was also p
> On Wed, 17 Jan 2024, Jan Hubicka wrote:
>
> > >
> > > I meant the new option might be named -fmin-function-alignment=
> > > rather than -falign-all-functions because of how it should
> > > override all other options.
> >
> > I was also pondering about both names. -falign-all-functions has the
On Wed, 17 Jan 2024, Jan Hubicka wrote:
> >
> > I meant the new option might be named -fmin-function-alignment=
> > rather than -falign-all-functions because of how it should
> > override all other options.
>
> I was also pondering about both names. -falign-all-functions has the
> advantage tha
>
> I meant the new option might be named -fmin-function-alignment=
> rather than -falign-all-functions because of how it should
> override all other options.
I was also pondering about both names. -falign-all-functions has the
advantage that it is similar to all the other alignment flags that a
On Wed, 17 Jan 2024, Jan Hubicka wrote:
> > > +falign-all-functions
> > > +Common Var(flag_align_all_functions) Optimization
> > > +Align the start of functions.
> >
> > all functions
> >
> > or maybe "of every function."?
>
> Fixed, thanks!
> > > +@opindex falign-all-functions=@var{n}
> > > +@
> > +falign-all-functions
> > +Common Var(flag_align_all_functions) Optimization
> > +Align the start of functions.
>
> all functions
>
> or maybe "of every function."?
Fixed, thanks!
> > +@opindex falign-all-functions=@var{n}
> > +@item -falign-all-functions
> > +Specify minimal alignment for f
gcc/ChangeLog:
>
> PR middle-end/88345
> * common.opt: Add -falign-all-functions
> * doc/invoke.texi: Add -falign-all-functions.
> (-falign-functions, -falign-labels, -falign-loops): Document
> that alignment is ignored in cold code.
>
) but it seems
to me that having separate flag is more readable.
Bootstrapped/regtested x86_64-linux, OK for master and later
backports to release branches?
gcc/ChangeLog:
PR middle-end/88345
* common.opt: Add -falign-all-functions
* doc/invoke.texi: Add -falign
