re PR debug/25023 (ICE in def_cfa_1, at dwarf2out.c:792)

	PR debug/25023
	* config/i386/i386.c (ix86_force_to_memory): Always use
	SImode push for HImode in -m32.
	(ix86_free_from_memory): Likewise.

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

From-SVN: r108537

diff --git a/gcc/ChangeLog b/gcc/ChangeLog
index 3831d49dfdbbcd987fb3a40dfd04a07d897d1e30..57cdbc34a84743126e7dee493ee089bb8d819b62 100644 (file)
--- a/gcc/ChangeLog
+++ b/gcc/ChangeLog
@@ -1,3 +1,10 @@
+2005-12-14  Jakub Jelinek  <jakub@redhat.com>
+
+       PR debug/25023
+       * config/i386/i386.c (ix86_force_to_memory): Always use
+       SImode push for HImode in -m32.
+       (ix86_free_from_memory): Likewise.
+
 2005-12-14  Richard Sandiford  <richard@codesourcery.com>
 
        * gcc/config/arm/arm.md: Provide a big-endian version of the

diff --git a/gcc/config/i386/i386.c b/gcc/config/i386/i386.c
index 33b9c6f7f15e291b7cbd5bcc2fdcdf7b01969eaf..74572188fdd67dc41689f5c0cdc45480d5fdf6a5 100644 (file)
--- a/gcc/config/i386/i386.c
+++ b/gcc/config/i386/i386.c
@@ -15790,9 +15790,8 @@ ix86_force_to_memory (enum machine_mode mode, rtx operand)
          }
          break;
        case HImode:
-         /* It is better to store HImodes as SImodes.  */
-         if (!TARGET_PARTIAL_REG_STALL)
-           operand = gen_lowpart (SImode, operand);
+         /* Store HImodes as SImodes.  */
+         operand = gen_lowpart (SImode, operand);
          /* FALLTHRU */
        case SImode:
          emit_insn (
@@ -15820,8 +15819,6 @@ ix86_free_from_memory (enum machine_mode mode)
 
       if (mode == DImode || TARGET_64BIT)
        size = 8;
-      else if (mode == HImode && TARGET_PARTIAL_REG_STALL)
-       size = 2;
       else
        size = 4;
       /* Use LEA to deallocate stack space.  In peephole2 it will be converted

diff --git a/gcc/testsuite/ChangeLog b/gcc/testsuite/ChangeLog
index dc707b3c990044bb1f024e73c7fc3021b3128e44..15f48b361e689475fe3c757581e4a2d7f81f50f9 100644 (file)
--- a/gcc/testsuite/ChangeLog
+++ b/gcc/testsuite/ChangeLog
@@ -1,3 +1,8 @@
+2005-12-14  Jakub Jelinek  <jakub@redhat.com>
+
+       PR debug/25023
+       * gcc.dg/pr25023.c: New test.
+
 2005-12-14  Kaveh R. Ghazi  <ghazi@caip.rutgers.edu>
 
        * g++.old-deja/g++.brendan/redecl1.C,

diff --git a/gcc/testsuite/gcc.dg/pr25023.c b/gcc/testsuite/gcc.dg/pr25023.c
new file mode 100644 (file)
index 0000000..b8cb7cf
--- /dev/null
+++ b/gcc/testsuite/gcc.dg/pr25023.c
@@ -0,0 +1,12 @@
+/* PR debug/25023 */
+/* { dg-do compile } */
+/* { dg-options "-O2" } */
+/* { dg-options "-O2 -mtune=i686" { target { { i?86-*-* || x86_64-*-* } && ilp32 } } } */
+
+extern unsigned char v;
+
+float
+foo (void)
+{
+  return v;
+}