"Junxian Zhu" <zhujunx...@oss.cipunited.com> writes:
> From: Junxian Zhu <zhujunx...@oss.cipunited.com>
>
> Add buildtime option to decide whether will compiler build with `-mmsa` 
> option default.
>
> gcc/ChangeLog:
>       * config.gcc: add -with-{no-}msa build option.
>       * config/mips/mips.h: Likewise.
>       * doc/install.texi: Likewise.

Thanks, pushed to trunk.  I think it's equivalent to adding an extra
mips*-*-* stanza and so suitable for stage 4.

Richard

>
> Signed-off-by: Junxian Zhu <zhujunx...@oss.cipunited.com>
> ---
>  gcc/config.gcc         | 19 +++++++++++++++++--
>  gcc/config/mips/mips.h |  3 ++-
>  gcc/doc/install.texi   |  8 ++++++++
>  3 files changed, 27 insertions(+), 3 deletions(-)
>
> diff --git a/gcc/config.gcc b/gcc/config.gcc
> index c070e6ecd2e..da3a6d3ba1f 100644
> --- a/gcc/config.gcc
> +++ b/gcc/config.gcc
> @@ -4709,7 +4709,7 @@ case "${target}" in
>               ;;
>  
>       mips*-*-*)
> -             supported_defaults="abi arch arch_32 arch_64 float fpu nan 
> fp_32 odd_spreg_32 tune tune_32 tune_64 divide llsc mips-plt synci lxc1-sxc1 
> madd4 compact-branches"
> +             supported_defaults="abi arch arch_32 arch_64 float fpu nan 
> fp_32 odd_spreg_32 tune tune_32 tune_64 divide llsc mips-plt synci lxc1-sxc1 
> madd4 compact-branches msa"
>  
>               case ${with_float} in
>               "" | soft | hard)
> @@ -4871,6 +4871,21 @@ case "${target}" in
>                       exit 1
>                       ;;
>               esac
> +
> +             case ${with_msa} in
> +             yes)
> +                     with_msa=msa
> +                     ;;
> +             no)
> +                     with_msa=no-msa
> +                     ;;
> +             "")
> +                     ;;
> +             *)
> +                     echo "Unknown msa type used in --with-msa" 1>&2
> +                     exit 1
> +                     ;;
> +             esac
>               ;;
>  
>       loongarch*-*-*)
> @@ -5815,7 +5830,7 @@ case ${target} in
>  esac
>  
>  t=
> -all_defaults="abi cpu cpu_32 cpu_64 arch arch_32 arch_64 tune tune_32 
> tune_64 schedule float mode fpu nan fp_32 odd_spreg_32 divide llsc mips-plt 
> synci tls lxc1-sxc1 madd4 isa_spec compact-branches"
> +all_defaults="abi cpu cpu_32 cpu_64 arch arch_32 arch_64 tune tune_32 
> tune_64 schedule float mode fpu nan fp_32 odd_spreg_32 divide llsc mips-plt 
> synci tls lxc1-sxc1 madd4 isa_spec compact-branches msa"
>  for option in $all_defaults
>  do
>       eval "val=\$with_"`echo $option | sed s/-/_/g`
> diff --git a/gcc/config/mips/mips.h b/gcc/config/mips/mips.h
> index fbb4372864f..13bc193b752 100644
> --- a/gcc/config/mips/mips.h
> +++ b/gcc/config/mips/mips.h
> @@ -916,7 +916,8 @@ struct mips_cpu_info {
>    {"synci", "%{!msynci:%{!mno-synci:-m%(VALUE)}}" },                 \
>    {"lxc1-sxc1", "%{!mlxc1-sxc1:%{!mno-lxc1-sxc1:-m%(VALUE)}}" }, \
>    {"madd4", "%{!mmadd4:%{!mno-madd4:-m%(VALUE)}}" }, \
> -  {"compact-branches", "%{!mcompact-branches=*:-mcompact-branches=%(VALUE)}" 
> } \
> +  {"compact-branches", "%{!mcompact-branches=*:-mcompact-branches=%(VALUE)}" 
> }, \
> +  {"msa", "%{!mmsa:%{!mno-msa:-m%(VALUE)}}" } \
>  
>  /* A spec that infers the:
>     -mnan=2008 setting from a -mips argument,
> diff --git a/gcc/doc/install.texi b/gcc/doc/install.texi
> index 8ef5c1414da..718f48fbaeb 100644
> --- a/gcc/doc/install.texi
> +++ b/gcc/doc/install.texi
> @@ -1653,6 +1653,14 @@ unfused is normally expected).  Disabling these 
> instructions is the
>  only way to ensure compatible code is generated; this will incur
>  a performance penalty.
>  
> +@item --with-msa
> +On MIPS targets, make @option{-mmsa} the default when no
> +@option{-mno-msa} option is passed.
> +
> +@item --without-msa
> +On MIPS targets, make @option{-mno-msa} the default when no
> +@option{-mmsa} option is passed. This is the default.
> +
>  @item --with-mips-plt
>  On MIPS targets, make use of copy relocations and PLTs.
>  These features are extensions to the traditional

Reply via email to