Jakub Jelinek <ja...@redhat.com> writes:
> Hi!
>
> The r13-2943-g11a113d501ff64 made aarch64.h include
> aarch64-option-extensions.def, but that file isn't installed
> for building plugins.
>
> The following patch should fix that, ok for trunk if it
> passes bootstrap/regtest + building plugin against it?
>
> 2022-12-20  Jakub Jelinek  <ja...@redhat.com>
>
>       * config/aarch64/t-aarch64 (OPTIONS_H_EXTRA): Add
>       aarch64-option-extensions.def.
>
> --- gcc/config/aarch64/t-aarch64.jj   2022-04-04 13:55:46.001615509 +0200
> +++ gcc/config/aarch64/t-aarch64      2022-12-20 11:31:03.245651809 +0100
> @@ -22,7 +22,8 @@ TM_H += $(srcdir)/config/aarch64/aarch64
>  OPTIONS_H_EXTRA += $(srcdir)/config/aarch64/aarch64-cores.def \
>                  $(srcdir)/config/aarch64/aarch64-arches.def \
>                  $(srcdir)/config/aarch64/aarch64-fusion-pairs.def \
> -                $(srcdir)/config/aarch64/aarch64-tuning-flags.def
> +                $(srcdir)/config/aarch64/aarch64-tuning-flags.def \
> +                $(srcdir)/config/aarch64/aarch64-option-extensions.def

Should this (and aarch64-fusion-pairs.def and aarch64-tuning-flags.def)
be in TM_H instead?  The first two OPTIONS_H_EXTRA entries seem to be
for aarch64-opt.h (included via aarch64.opt).

I guess TM_H should also have aarch64-arches.def, since it's included
for aarch64_feature.

OK with that change if it works/makes sense.

Thanks,
Richard

>  
>  $(srcdir)/config/aarch64/aarch64-tune.md: s-aarch64-tune-md; @true
>  s-aarch64-tune-md: $(srcdir)/config/aarch64/gentune.sh \
>
>       Jakub

Reply via email to