[PATCH] aarch64: disable feature b16b16

[Richard Earnshaw (lists)]

On 12/07/2024 09:02, yury.khrustalev@arm.com wrote:
> From: Matthieu Longo <Matthieu.Longo@arm.com>
> 
> Feature b16b16 is currently incomplete and requires re-work.
> 
> This patch disables the command line option for b16b16, and marks the
> associated tests as XFAIL.
> 
> ---
> Feature b16b16 is currently incomplete and requires re-work.
> This patch disables the command line option for b16b16, and marks the associated tests as XFAIL.
> 
> Regression tested on aarch64-unknown-linux-gnu, and no regression found.
> 
> Ok for binutils-master?

I've made one small change to this and pushed it.  Conventionally we write 'xfail *-*-*' to match any triplet rather than just 'xfail *'.  I've also amended the commit message slightly, to remove 'This patch', with corresponding adjustments to the grammar in the rest of the sentence.

R.

> 
> Regards,
> Yury
> ---
>  gas/config/tc-aarch64.c                        | 4 +++-
>  gas/doc/c-aarch64.texi                         | 4 ++--
>  gas/testsuite/gas/aarch64/bfloat16-1.d         | 1 +
>  gas/testsuite/gas/aarch64/bfloat16-2-invalid.d | 1 +
>  4 files changed, 7 insertions(+), 3 deletions(-)
> 
> diff --git a/gas/config/tc-aarch64.c b/gas/config/tc-aarch64.c
> index 8690b8f9142..ed5ab762bc9 100644
> --- a/gas/config/tc-aarch64.c
> +++ b/gas/config/tc-aarch64.c
> @@ -10705,7 +10705,9 @@ static const struct aarch64_option_cpu_value_table aarch64_features[] = {
>    {"rasv2",		AARCH64_FEATURE (RASv2), AARCH64_FEATURE (RAS)},
>    {"ite",		AARCH64_FEATURE (ITE), AARCH64_NO_FEATURES},
>    {"d128",		AARCH64_FEATURE (D128), D128_FEATURE_DEPS},
> -  {"b16b16",		AARCH64_FEATURE (B16B16), AARCH64_FEATURE (SVE2)},
> +  // Feature b16b16 is currently incomplete.
> +  // TODO: finish implementation and enable relevant flags.
> +  //{"b16b16",		AARCH64_FEATURE (B16B16), AARCH64_FEATURE (SVE2)},
>    {"sme2p1",		AARCH64_FEATURE (SME2p1), AARCH64_FEATURE (SME2)},
>    {"sve2p1",		AARCH64_FEATURE (SVE2p1), AARCH64_FEATURE (SVE2)},
>    {"rcpc3",		AARCH64_FEATURE (RCPC3), AARCH64_FEATURE (RCPC2)},
> diff --git a/gas/doc/c-aarch64.texi b/gas/doc/c-aarch64.texi
> index 212630477a1..8e46038a787 100644
> --- a/gas/doc/c-aarch64.texi
> +++ b/gas/doc/c-aarch64.texi
> @@ -161,8 +161,8 @@ automatically cause those extensions to be disabled.
>  @headitem Extension @tab Depends upon @tab Description
>  @item @code{aes} @tab @code{simd}
>   @tab Enable the AES and PMULL cryptographic extensions.
> -@item @code{b16b16} @tab @code{sve2}
> - @tab Enable BFloat16 to BFloat16 arithmetic for SVE2 and SME2.
> +@c @item @code{b16b16} @tab @code{sve2}
> +@c  @tab Enable BFloat16 to BFloat16 arithmetic for SVE2 and SME2.
>  @item @code{bf16} @tab @code{fp}
>   @tab Enable BFloat16 extension.
>  @item @code{brbe} @tab
> diff --git a/gas/testsuite/gas/aarch64/bfloat16-1.d b/gas/testsuite/gas/aarch64/bfloat16-1.d
> index 51f7e6cab20..9027f3c9e36 100644
> --- a/gas/testsuite/gas/aarch64/bfloat16-1.d
> +++ b/gas/testsuite/gas/aarch64/bfloat16-1.d
> @@ -1,6 +1,7 @@
>  #name: Test of SVE2.1 and SME2.1 non-widening BFloat16 instructions.
>  #as: -march=armv9.4-a+b16b16
>  #objdump: -dr
> +#xfail: *
>  
>  [^:]+:     file format .*
>  
> diff --git a/gas/testsuite/gas/aarch64/bfloat16-2-invalid.d b/gas/testsuite/gas/aarch64/bfloat16-2-invalid.d
> index 1cd27454d42..8a9a56b10e5 100644
> --- a/gas/testsuite/gas/aarch64/bfloat16-2-invalid.d
> +++ b/gas/testsuite/gas/aarch64/bfloat16-2-invalid.d
> @@ -2,3 +2,4 @@
>  #as: -march=armv9.4-a+b16b16
>  #source: bfloat16-2-invalid.s
>  #error_output: bfloat16-2-invalid.l
> +#xfail: *