[1/3] aarch64: Fix overly lax +frintts dependency

Message ID d6ab75f8-6175-3cdf-a960-65c39215d887@e124511.cambridge.arm.com
State New
Headers
Series aarch64: Assorted dependency fixes |

Checks

Context Check Description
linaro-tcwg-bot/tcwg_binutils_build--master-arm fail Patch failed to apply
linaro-tcwg-bot/tcwg_binutils_build--master-aarch64 fail Patch failed to apply

Commit Message

Andrew Carlotti Jan. 31, 2025, 1:17 p.m. UTC
  We agreed with LLVM that +frintts should only enable +fp, not +simd.
This also matches the dependency used in GCC.
  

Patch

diff --git a/gas/config/tc-aarch64.c b/gas/config/tc-aarch64.c
index 8b074ff22dd6870fb1cc7d796015832860c37293..b79af4044ffe904472beddfa4f8400d9f67b433c 100644
--- a/gas/config/tc-aarch64.c
+++ b/gas/config/tc-aarch64.c
@@ -10695,7 +10695,7 @@  static const struct aarch64_option_cpu_value_table aarch64_features[] = {
   {"rcpc2",		AARCH64_FEATURE (RCPC2), AARCH64_FEATURE (RCPC)},
   {"dotprod",		AARCH64_FEATURE (DOTPROD), AARCH64_FEATURE (SIMD)},
   {"sha2",		AARCH64_FEATURE (SHA2), AARCH64_FEATURE (SIMD)},
-  {"frintts",		AARCH64_FEATURE (FRINTTS), AARCH64_FEATURE (SIMD)},
+  {"frintts",		AARCH64_FEATURE (FRINTTS), AARCH64_FEATURE (FP)},
   {"sb",		AARCH64_FEATURE (SB), AARCH64_NO_FEATURES},
   {"predres",		AARCH64_FEATURE (PREDRES), AARCH64_NO_FEATURES},
   {"predres2",		AARCH64_FEATURE (PREDRES2), AARCH64_FEATURE (PREDRES)},
diff --git a/gas/doc/c-aarch64.texi b/gas/doc/c-aarch64.texi
index 76f65d7c3f9391669d7bcf0e4de767cfd40bc902..5d0aac8fe0ae31f4f72177c5bdc55adb4e0e2daa 100644
--- a/gas/doc/c-aarch64.texi
+++ b/gas/doc/c-aarch64.texi
@@ -205,7 +205,7 @@  automatically cause those extensions to be disabled.
  @tab Enable Armv8.2 16-bit floating-point multiplication variant support.
 @item @code{fp16} @tab @code{fp}
  @tab Enable Armv8.2 16-bit floating-point support.
-@item @code{frintts} @tab @code{simd}
+@item @code{frintts} @tab @code{fp}
  @tab Enable floating-point round to integral value instructions.
 @item @code{gcs} @tab
  @tab Enable the Guarded Control Stack Extension.