[PATCH] newlib: delete (most) redundant lib_a_CCASFLAGS=$(AM_CCASFLAGS)

Corinna Vinschen vinschen@redhat.com
Tue Jan 18 10:00:59 GMT 2022


On Jan 17 04:39, Mike Frysinger wrote:
> Since automake already sets per-library CCASFLAGS to $(AM_CCASFLAGS)
> by default, there's no need to explicitly set it here.
> 
> Many of these dirs don't have .S files in the first place, so the rule
> doesn't even do anything.  That can easily be seen when Makefile.in has
> no changes as a result.
> 
> For the dirs with .S files, the custom rules are the same as the pattern
> .S.o rules, so this is a nice cleanup.
> 
> The only dir that was adding extra flags (newlib/libc/machine/mn10300/)
> to the per-library setting can have it moved to the global AM_CCASFLAGS
> since the subdir only has one target.  Although the setting just adds
> extra debugging flags, so maybe it should be deleted in general.
> 
> There are a few dirs that we leave the redundant setting in place.  This
> is to workaround an automake limitation in subdirs that support building
> with & w/out libtool:
> https://www.gnu.org/software/automake/manual/html_node/Objects-created-both-with-libtool-and-without.html
> ---
> NB: Makefile.in generated output has been filtered.

LGTM, please push.


Thanks,
Corinna



More information about the Newlib mailing list