Hi,

this replaces the ICE by a sorry message for the use of reverse scalar storage 
order with a 128-bit decimal floating-point type on 32-bit platforms.

Tested on x86-64/Linux, OK for the mainline?


2020-12-03  Eric Botcazou  <ebotca...@adacore.com>

        * expmed.c (flip_storage_order): In the case of a non-integer mode,
        sorry out if the integer mode to be used instead is not supported.


2020-12-03  Eric Botcazou  <ebotca...@adacore.com>

        * gcc.dg/pr98099.c: New test.

-- 
Eric Botcazou
/* PR middle-end/98099 */
/* Reported by G. Steinmetz <gs...@t-online.de> */

/* { dg-do compile } */
/* { dg-options "-fsso-struct=big-endian" } */

struct S { _Decimal128 a; };

_Decimal128 f (struct S x)
{
  return x.a; /* { dg-message "sorry, unimplemented: reverse storage order" "" { target { ! int128 } } } */
}
diff --git a/gcc/expmed.c b/gcc/expmed.c
index ce88f322961..0d600bd7b54 100644
--- a/gcc/expmed.c
+++ b/gcc/expmed.c
@@ -412,7 +412,8 @@ flip_storage_order (machine_mode mode, rtx x)
 	  && __builtin_expect (reverse_float_storage_order_supported < 0, 0))
 	check_reverse_float_storage_order_support ();
 
-      if (!int_mode_for_size (GET_MODE_PRECISION (mode), 0).exists (&int_mode))
+      if (!int_mode_for_size (GET_MODE_PRECISION (mode), 0).exists (&int_mode)
+	  || !targetm.scalar_mode_supported_p (int_mode))
 	{
 	  sorry ("reverse storage order for %smode", GET_MODE_NAME (mode));
 	  return x;

Reply via email to