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 that it is similar to all the other alignment flags that are
> all called -falign-XXX
>
> but both options are finte for me.
> >
> > Otherwise is there an updated patch to look at?
>
> I will prepare one. So shall I drop the max-skip support for alignment
> and rename the flag?
Yes.
> Honza
> >
> > Richard.
> >
> > > > -flimit-function-alignment should not have an effect on it
> > > > and even very small functions should be aligned.
> > >
> > > I write that it is not affected by limit-function-alignment
> > > @opindex falign-all-functions=@var{n}
> > > @item -falign-all-functions
> > > Specify minimal alignment for function entry. Unlike
> > > @option{-falign-functions}
> > > this alignment is applied also to all functions (even those considered
> > > cold).
> > > The alignment is also not affected by @option{-flimit-function-alignment}
> > >
> > > Because indeed that would break the atomicity of updates.
> >
> >
> >
> > > Honza
> > > >
> > > > Richard.
> > > >
> > > > > + }
> > > > > +
> > > > > /* Handle a user-specified function alignment.
> > > > > Note that we still need to align to DECL_ALIGN, as above,
> > > > > because ASM_OUTPUT_MAX_SKIP_ALIGN might not do any alignment at
> > > > > all. */
> > > > >
> > > >
> > > > --
> > > > Richard Biener <rguent...@suse.de>
> > > > SUSE Software Solutions Germany GmbH,
> > > > Frankenstrasse 146, 90461 Nuernberg, Germany;
> > > > GF: Ivo Totev, Andrew McDonald, Werner Knoblich; (HRB 36809, AG
> > > > Nuernberg)
> > >
> >
> > --
> > Richard Biener <rguent...@suse.de>
> > SUSE Software Solutions Germany GmbH,
> > Frankenstrasse 146, 90461 Nuernberg, Germany;
> > GF: Ivo Totev, Andrew McDonald, Werner Knoblich; (HRB 36809, AG Nuernberg)
>
--
Richard Biener <rguent...@suse.de>
SUSE Software Solutions Germany GmbH,
Frankenstrasse 146, 90461 Nuernberg, Germany;
GF: Ivo Totev, Andrew McDonald, Werner Knoblich; (HRB 36809, AG Nuernberg)
