Re: [PATCH 1/1] gdb, fortran: Fix quad floating-point type for ifort compiler.

[Tom Tromey <tom@tromey.com>]

> When examining memory for this Fortran test, I see 128-bit values in
> binary128 format, so I assume that this is just a software/compiler feature. 
> There is also decimal128 support in e.g. gcc.  I tried it only briefly,
> we don't end up in i386_floatformat_for_type for those types, but
> GDB prints them just fine.  I assume as they are marked as
> DW_ATE_decimal_float instead of DW_ATE_float, but I didn't look into 
> it in detail.

> I am not aware of any other float types that exist with this length.

I wonder, then, if this patch would work?
This way we would not have to curate a list of names.

Tom

diff --git a/gdb/i386-tdep.c b/gdb/i386-tdep.c
index 476d17f1efd..57e50ffdd73 100644
--- a/gdb/i386-tdep.c
+++ b/gdb/i386-tdep.c
@@ -8277,20 +8277,8 @@ static const struct floatformat **
 i386_floatformat_for_type (struct gdbarch *gdbarch,
 			   const char *name, int len)
 {
-  if (len == 128 && name)
-    if (strcmp (name, "__float128") == 0
-	|| strcmp (name, "_Float128") == 0
-	|| strcmp (name, "complex _Float128") == 0
-	|| strcmp (name, "complex(kind=16)") == 0
-	|| strcmp (name, "COMPLEX(16)") == 0
-	|| strcmp (name, "complex*32") == 0
-	|| strcmp (name, "COMPLEX*32") == 0
-	|| strcmp (name, "quad complex") == 0
-	|| strcmp (name, "real(kind=16)") == 0
-	|| strcmp (name, "real*16") == 0
-	|| strcmp (name, "REAL*16") == 0
-	|| strcmp (name, "REAL(16)") == 0)
-      return floatformats_ieee_quad;
+  if (len == 128)
+    return floatformats_ieee_quad;
 
   return default_floatformat_for_type (gdbarch, name, len);
 }