[Bug target/31344] New: [4.3 regression] Bootstrap failure to compile decLibrary.c

[Bug target/31344] New: [4.3 regression] Bootstrap failure to compile decLibrary.c

[fxcoudert at gcc dot gnu dot org]

My nightly bootstrap of mainline on i386-linux failed tonight, on revision
123192, with:

/home/fxcoudert/gfortran_nightbuild/trunk/libgcc/../libdecnumber/decLibrary.c:
In function ?isinfd64?:
/home/fxcoudert/gfortran_nightbuild/trunk/libgcc/../libdecnumber/decLibrary.c:65:
error: unrecognizable insn:
(insn 11 10 12 3
/home/fxcoudert/gfortran_nightbuild/trunk/libgcc/../libdecnumber/decLibrary.c:62
(set (reg/f:SI 61)
        (pre_dec:SI (reg/f:SI 7 sp))) -1 (nil)
    (nil))
/home/fxcoudert/gfortran_nightbuild/trunk/libgcc/../libdecnumber/decLibrary.c:65:
internal compiler error: in extract_insn, at recog.c:2119

The last revision known to compile OK on that particular setup was: 123178. I
can reproduce it with the attached preprocessed source and the following
command line:

$ /home/fxcoudert/gfortran_nightbuild/ibin-20070325/./gcc/cc1 -fpreprocessed
foo.i -mtune=i386
 gnu_dev_major gnu_dev_minor gnu_dev_makedev __sigismember __sigaddset
__sigdelset isinfd32
/home/fxcoudert/gfortran_nightbuild/trunk/libgcc/../libdecnumber/decLibrary.c:
In function ?isinfd32?:
/home/fxcoudert/gfortran_nightbuild/trunk/libgcc/../libdecnumber/decLibrary.c:54:
error: unrecognizable insn:
(insn 10 9 11 3 (set (reg/f:SI 64)
        (pre_dec:SI (reg/f:SI 7 sp))) -1 (nil)
    (nil))
/home/fxcoudert/gfortran_nightbuild/trunk/libgcc/../libdecnumber/decLibrary.c:54:
internal compiler error: in extract_insn, at recog.c:2119

With -mtune=i[345]86, an ICE occurs, while for -mtune=i686 there is no problem.


-- 
           Summary: [4.3 regression] Bootstrap failure to compile
                    decLibrary.c
           Product: gcc
           Version: 4.3.0
            Status: UNCONFIRMED
          Keywords: build
          Severity: blocker
          Priority: P3
         Component: target
        AssignedTo: unassigned at gcc dot gnu dot org
        ReportedBy: fxcoudert at gcc dot gnu dot org
 GCC build triplet: i386-pc-linux-gnu
  GCC host triplet: i386-pc-linux-gnu
GCC target triplet: i386-pc-linux-gnu


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=31344

[Bug target/31344] [4.3 regression] Bootstrap failure to compile decLibrary.c

[pinskia at gcc dot gnu dot org]

-- 

pinskia at gcc dot gnu dot org changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |pinskia at gcc dot gnu dot
                   |                            |org
           Keywords|                            |ice-on-valid-code
   Target Milestone|---                         |4.3.0


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=31344

[Bug target/31344] [4.3 regression] Bootstrap failure to compile decLibrary.c

[pinskia at gcc dot gnu dot org]

------- Comment #1 from pinskia at gcc dot gnu dot org  2007-03-25 18:58 -------
Note, I think decLibrary.c changed but does not cause this not to be an user
visible regression.


-- 


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=31344

[Bug target/31344] [4.3 regression] Bootstrap failure to compile decLibrary.c

[hjl at lucon dot org]

------- Comment #2 from hjl at lucon dot org  2007-03-25 19:18 -------
Where is "the attached preprocessed source"?


-- 

hjl at lucon dot org changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |hjl at lucon dot org


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=31344

[Bug target/31344] [4.3 regression] Bootstrap failure to compile decLibrary.c

[fxcoudert at gcc dot gnu dot org]

------- Comment #3 from fxcoudert at gcc dot gnu dot org  2007-03-25 19:23 -------
Created an attachment (id=13285)
 --> (http://gcc.gnu.org/bugzilla/attachment.cgi?id=13285&action=view)
Preprocessed source file triggering the ICE

Here's the preprocessed source, sorry for forgetting it in the first place.


-- 


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=31344

[Bug target/31344] [4.3 regression] Bootstrap failure to compile decLibrary.c

[fxcoudert at gcc dot gnu dot org]

------- Comment #4 from fxcoudert at gcc dot gnu dot org  2007-03-25 19:38 -------
Reduced testcase:

$ cat a.c
typedef struct
{
  unsigned char bits;
} decNumber;

typedef struct
{
  unsigned char bytes[1];
} decimal32;

decNumber *__decimal32ToNumber (const decimal32 *, decNumber *);
void __host_to_ieee_32 (_Decimal32, decimal32 *);
extern int isinfd32 (_Decimal32);

int
isinfd32 (_Decimal32 arg)
{
  decNumber dn;
  decimal32 d32;
  __host_to_ieee_32 (arg, &d32);
  __decimal32ToNumber (&d32, &dn);
}
$ ./cc1 a.i -mtune=i386
 isinfd32
/home/fxcoudert/foo.i: In function ?isinfd32?:
/home/fxcoudert/foo.i:22: error: unrecognizable insn:
(insn 10 9 11 3 (set (reg/f:SI 60)
        (pre_dec:SI (reg/f:SI 7 sp))) -1 (nil)
    (nil))
/home/fxcoudert/foo.i:22: internal compiler error: in extract_insn, at
recog.c:2119


-- 


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=31344

[Bug target/31344] [4.3 regression] Bootstrap failure to compile decLibrary.c

[hjl at lucon dot org]

[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #1: Type: text/plain, Size: 675 bytes --]



------- Comment #5 from hjl at lucon dot org  2007-03-26 01:18 -------
It also happens with -Os:

bash-3.1$ /export/build/gnu/gcc/build-x86_64-linux/stage1-gcc/xgcc
-B/export/build/gnu/gcc/build-x86_64-linux/stage1-gcc/ -m32  -S -o bar.s bar.i 
-Os 
bar.i: In function ‘isinfd32’:
bar.i:22: error: unrecognizable insn:
(insn 11 10 12 3 (set (reg/f:SI 61)
        (pre_dec:SI (reg/f:SI 7 sp))) -1 (nil)
    (nil))
bar.i:22: internal compiler error: in extract_insn, at recog.c:2119
Please submit a full bug report,
with preprocessed source if appropriate.
See <URL:http://gcc.gnu.org/bugs.html> for instructions.


-- 


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=31344

[Bug target/31344] [4.3 regression] Bootstrap failure to compile decLibrary.c

[hjl at lucon dot org]

------- Comment #6 from hjl at lucon dot org  2007-03-26 01:55 -------
Jan, it may be related to your code:

http://gcc.gnu.org/ml/gcc-patches/2000-04/msg00263.html

Any idea?


-- 

hjl at lucon dot org changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |hubicka at gcc dot gnu dot
                   |                            |org


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=31344

[Bug target/31344] [4.3 regression] Bootstrap failure to compile decLibrary.c

[hjl at lucon dot org]

------- Comment #7 from hjl at lucon dot org  2007-03-26 03:15 -------
It looks like expr.c and i386.c don't agree how to generate push
instuction. i386.c has gen_push and expr.c has emit_single_push_insn.
If emit_single_push_insn is called, it will generate invalid patten:

(insn 10 9 11 3 (set (reg/f:SI 64)
        (pre_dec:SI (reg/f:SI 7 sp))) -1 (nil)
    (nil))

from emit_single_push_insn in expr.c

3519      if (GET_MODE_SIZE (mode) == rounded_size)
3520        dest_addr = gen_rtx_fmt_e (STACK_PUSH_CODE, Pmode,
stack_pointer_rtx);


-- 


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=31344

[Bug target/31344] [4.3 regression] Bootstrap failure to compile decLibrary.c

[hjl at lucon dot org]

------- Comment #8 from hjl at lucon dot org  2007-03-26 04:04 -------
The invalid insn is added via

#0  add_insn (insn=0x2aaaaaafc690)
    at /export/gnu/src/gcc/gcc/gcc/emit-rtl.c:3411
#1  0x00000000004fb88d in emit_insn (x=0x2aaaaaafc690)
    at /export/gnu/src/gcc/gcc/gcc/emit-rtl.c:4336
#2  0x000000000051f3f8 in emit_move_insn_1 (x=0x2aaaaacecd80, y=0x2aaaaacfb080)
    at /export/gnu/src/gcc/gcc/gcc/expr.c:3260
#3  0x000000000051f725 in emit_move_insn (x=0x2aaaaacecd80, y=0x2aaaaacfb080)
    at /export/gnu/src/gcc/gcc/gcc/expr.c:3351
#4  0x0000000000507baf in force_reg (mode=SImode, x=0x2aaaaacfb080)
    at /export/gnu/src/gcc/gcc/gcc/explow.c:680
#5  0x0000000000507695 in memory_address (mode=SImode, x=0x2aaaaacfb080)
    at /export/gnu/src/gcc/gcc/gcc/explow.c:485
#6  0x00000000004f7491 in change_address_1 (memref=0x2aaaaacecd60, 
    mode=SImode, addr=0x2aaaaacfb080, validate=1)
    at /export/gnu/src/gcc/gcc/gcc/emit-rtl.c:1811
#7  0x00000000004f7add in adjust_address_1 (memref=0x2aaaaacecd60, 
    mode=SImode, offset=0, validate=1, adjust=1)
    at /export/gnu/src/gcc/gcc/gcc/emit-rtl.c:1895
#8  0x000000000051e778 in emit_move_change_mode (new_mode=SImode, 
    old_mode=SDmode, x=0x2aaaaacecd60, force=1 '\001')
    at /export/gnu/src/gcc/gcc/gcc/expr.c:2882
#9  0x000000000051e830 in emit_move_via_integer (mode=SDmode, 
    x=0x2aaaaacecd60, y=0x2aaaaacecb40, force=1 '\001')
---Type <return> to continue, or q <return> to quit---
    at /export/gnu/src/gcc/gcc/gcc/expr.c:2921
#10 0x000000000051f45f in emit_move_insn_1 (x=0x2aaaaacecd60, y=0x2aaaaacecb40)
    at /export/gnu/src/gcc/gcc/gcc/expr.c:3268
#11 0x000000000051f725 in emit_move_insn (x=0x2aaaaacecd60, y=0x2aaaaacecb40)
    at /export/gnu/src/gcc/gcc/gcc/expr.c:3351
#12 0x000000000051fecc in emit_single_push_insn (mode=SDmode, 
    x=0x2aaaaacecb40, type=0x2aaaaab16c00)
    at /export/gnu/src/gcc/gcc/gcc/expr.c:3582
#13 0x0000000000520757 in emit_push_insn (x=0x2aaaaacecb40, mode=SDmode, 
    type=0x2aaaaab16c00, size=0x0, align=32, partial=0, reg=0x0, extra=0, 
    args_addr=0x0, args_so_far=0x2aaaaaafb400, reg_parm_stack_space=0, 
    alignment_pad=0x2aaaaaafb400) at /export/gnu/src/gcc/gcc/gcc/expr.c:3852
#14 0x00000000004a3c1c in store_one_arg (arg=0x7fff5d787aa8, argblock=0x0, 
    flags=0, variable_size=0, reg_parm_stack_space=0)
    at /export/gnu/src/gcc/gcc/gcc/calls.c:4239
#15 0x000000000049f94c in expand_call (exp=0x2aaaaacebb40, target=0x0, 
    ignore=1) at /export/gnu/src/gcc/gcc/gcc/calls.c:2652
#16 0x000000000052a40c in expand_expr_real_1 (exp=0x2aaaaacebb40, target=0x0, 
    tmode=VOIDmode, modifier=EXPAND_NORMAL, alt_rtl=0x0)
    at /export/gnu/src/gcc/gcc/gcc/expr.c:7785
#17 0x0000000000527414 in expand_expr_real (exp=0x2aaaaacebb40, 
    target=0x2aaaaaafb400, tmode=VOIDmode, modifier=EXPAND_NORMAL, alt_rtl=0x0)
    at /export/gnu/src/gcc/gcc/gcc/expr.c:6785

We didn't see this problem before is because decimal float wasn't enabled
by default on x86 until revision 123185.


-- 


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=31344

[Bug target/31344] [4.3 regression] Bootstrap failure to compile decLibrary.c

[fxcoudert at gcc dot gnu dot org]

-- 

fxcoudert at gcc dot gnu dot org changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|UNCONFIRMED                 |NEW
     Ever Confirmed|0                           |1
   Last reconfirmed|0000-00-00 00:00:00         |2007-03-26 11:55:08
               date|                            |


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=31344

[Bug target/31344] [4.3 regression] Bootstrap failure to compile decLibrary.c

[hjl at lucon dot org]

------- Comment #9 from hjl at lucon dot org  2007-03-26 18:37 -------
Any tuning target doesn't set x86_accumulate_outgoing_args will fail.


-- 


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=31344

[Bug target/31344] [4.3] dfp isn't properly supported on x86

[hjl at lucon dot org]

------- Comment #10 from hjl at lucon dot org  2007-03-26 19:30 -------
I think dfp should be disabled by default on x86. We are missing patterns
for "movsd", "movdd", and maybe "movtd".


-- 

hjl at lucon dot org changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
            Summary|[4.3 regression] Bootstrap  |[4.3] dfp isn't properly
                   |failure to compile          |supported on x86
                   |decLibrary.c                |


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=31344

[Bug target/31344] [4.3 regression] Bootstrap failure to compile decLibrary.c

[pinskia at gcc dot gnu dot org]

------- Comment #11 from pinskia at gcc dot gnu dot org  2007-03-26 19:39 -------
The ICE is still a bug which needs to be fixed Plus this is still a regression.


-- 

pinskia at gcc dot gnu dot org changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
            Summary|[4.3] dfp isn't properly    |[4.3 regression] Bootstrap
                   |supported on x86            |failure to compile
                   |                            |decLibrary.c


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=31344

[Bug target/31344] [4.2/4.3]: DFP isn't properly supported on x86

[hjl at lucon dot org]

------- Comment #12 from hjl at lucon dot org  2007-03-26 19:56 -------
(In reply to comment #11)
> The ICE is still a bug which needs to be fixed Plus this is still a regression.
> 

Failing to compile this code isn't a regression since the same code always
fails with gcc 4.2 and 4.3 when DFP is enabled. The difference is DFP was
disabled by default before. That is why people didn't see it. Now DFP is
enabled by default on x86 and people start seeing it.


-- 

hjl at lucon dot org changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
            Summary|[4.3 regression] Bootstrap  |[4.2/4.3]: DFP  isn't
                   |failure to compile          |properly supported on x86
                   |decLibrary.c                |


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=31344

[Bug target/31344] [4.3 Regression]: DFP isn't properly supported on x86

[pinskia at gcc dot gnu dot org]

------- Comment #13 from pinskia at gcc dot gnu dot org  2007-03-26 20:04 -------
The build failure is still a regression, no matter which you look at it.


-- 

pinskia at gcc dot gnu dot org changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
            Summary|[4.2/4.3]: DFP  isn't       |[4.3 Regression]: DFP  isn't
                   |properly supported on x86   |properly supported on x86


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=31344

[Bug target/31344] [4.3 Regression]: DFP isn't properly supported on x86

[hjl at lucon dot org]

------- Comment #14 from hjl at lucon dot org  2007-03-26 20:41 -------
This isn't a real patch. But it seems to work on the testcase.

--- i386.md.dfp 2007-03-24 12:21:45.000000000 -0700
+++ i386.md     2007-03-26 12:30:21.000000000 -0700
@@ -1850,6 +1850,59 @@
   ""
   "ix86_expand_move (DImode, operands); DONE;")

+(define_expand "movsd"
+  [(set (match_operand:SD 0 "nonimmediate_operand" "")
+       (match_operand:SD 1 "general_operand" ""))]
+  ""
+  "ix86_expand_move (SDmode, operands); DONE;")
+
+(define_insn "*pushsd2"
+  [(set (match_operand:SD 0 "push_operand" "=<")
+       (match_operand:SD 1 "general_no_elim_operand" "ri*m"))]
+  "!TARGET_64BIT"
+  "push{l}\t%1"
+  [(set_attr "type" "push")
+   (set_attr "mode" "SI")])
+
+;; For 64BIT abi we always round up to 8 bytes.
+(define_insn "*pushsd2_rex64"
+  [(set (match_operand:SD 0 "push_operand" "=X")
+       (match_operand:SD 1 "nonmemory_no_elim_operand" "ri"))]
+  "TARGET_64BIT"
+  "push{q}\t%q1"
+  [(set_attr "type" "push")
+   (set_attr "mode" "SI")])
+
+(define_expand "movdd"
+  [(set (match_operand:DD 0 "nonimmediate_operand" "")
+       (match_operand:DD 1 "general_operand" ""))]
+  ""
+  "ix86_expand_move (DDmode, operands); DONE;")
+
+(define_insn "*pushdd"
+  [(set (match_operand:DD 0 "push_operand" "=<")
+       (match_operand:DD 1 "general_no_elim_operand" "riF*m"))]
+  "!TARGET_64BIT"
+  "#")
+
+(define_insn "*pushdd2_rex64"
+  [(set (match_operand:DD 0 "push_operand" "=<,!<")
+       (match_operand:DD 1 "general_no_elim_operand" "re*m,n"))]
+  "TARGET_64BIT"
+  "@
+   push{q}\t%1
+   #"
+  [(set_attr "type" "push,multi")
+   (set_attr "mode" "DI")])
+
+(define_split
+  [(set (match_operand:DD 0 "push_operand" "")
+        (match_operand:DD 1 "general_operand" ""))]
+  "!TARGET_64BIT && reload_completed
+   && (! MMX_REG_P (operands[1]) && !SSE_REG_P (operands[1]))"
+  [(const_int 0)]
+  "ix86_split_long_move (operands); DONE;")
+
 (define_insn "*pushdi"
   [(set (match_operand:DI 0 "push_operand" "=<")
        (match_operand:DI 1 "general_no_elim_operand" "riF*m"))]


-- 


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=31344

[Bug bootstrap/31344] [4.3 Regression] bootstrap broken on i[345]86-linux

[fxcoudert at gcc dot gnu dot org]

------- Comment #15 from fxcoudert at gcc dot gnu dot org  2007-03-31 16:01 -------
This is a bootstrap breakage, and it's was reported almost a week ago. Can you
please test and commit the trivial fix you suggested (disable decimal float for
on i386-linux)? Then you'll have all time to investigate why the DFP code fails
and other stuff.

Index: configure.ac
===================================================================
--- configure.ac        (revision 123378)
+++ configure.ac        (working copy)
@@ -680,7 +680,7 @@
 ],
 [
   case $target in
-    powerpc*-*-linux* | i?86*-*-linux* | x86_64*-*-linux*)
+    powerpc*-*-linux* | i686*-*-linux* | x86_64*-*-linux*)
       enable_decimal_float=yes
       ;;
     *)


-- 

fxcoudert at gcc dot gnu dot org changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
          Component|target                      |bootstrap
      Known to fail|                            |4.3.0
   Last reconfirmed|2007-03-26 11:55:08         |2007-03-31 16:01:43
               date|                            |
            Summary|[4.3 Regression]: DFP  isn't|[4.3 Regression] bootstrap
                   |properly supported on x86   |broken on i[345]86-linux


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=31344

[Bug bootstrap/31344] [4.3 Regression] bootstrap broken on i[345]86-linux

[hjl at lucon dot org]

------- Comment #16 from hjl at lucon dot org  2007-04-03 23:22 -------
A patch is posted at

http://gcc.gnu.org/ml/gcc-patches/2007-03/msg01764.html


-- 


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=31344

[Bug bootstrap/31344] [4.3 Regression] bootstrap broken on i[345]86-linux

[pinskia at gcc dot gnu dot org]

------- Comment #17 from pinskia at gcc dot gnu dot org  2007-04-14 08:38 -------
*** Bug 31571 has been marked as a duplicate of this bug. ***


-- 

pinskia at gcc dot gnu dot org changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |debian-gcc at lists dot
                   |                            |debian dot org


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=31344

[Bug bootstrap/31344] [4.3 Regression] bootstrap broken on i[345]86-linux

[hjl at lucon dot org]

------- Comment #18 from hjl at lucon dot org  2007-04-30 17:45 -------
An updated patch is posted at

http://gcc.gnu.org/ml/gcc-patches/2007-04/msg02003.html


-- 


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=31344

[Bug bootstrap/31344] [4.3 Regression] bootstrap broken on i[345]86-linux

[ubizjak at gmail dot com]

------- Comment #19 from ubizjak at gmail dot com  2007-05-07 08:19 -------
Here is the problem:

Compilation enteres emit_move_via_integer() with:
x = (mem/i:SD (pre_dec:SI (reg/f:SI 7 sp)) [0 S4 A32])
y = (reg/v:SD 59 [ arg ])

emit_move_change_mode() generates invalid insn from x because it forces
pre_dec:SI into pseudo register. emit_move_change_mode() generates following
sequence:

(insn 11 10 12 (set (reg/f:SI 61)
        (pre_dec:SI (reg/f:SI 7 sp))) -1 (nil)
    (nil))

(insn 12 11 13 (set (mem/i:SI (reg/f:SI 61) [0 S4 A32])
        (subreg:SI (reg/v:SD 59 [ arg ]) 0)) -1 (nil)
    (nil))

For some reason, pre_dec is forced into pseudo in
adjust_address_1()/change_address_1(). Above sequence is surely wrong, as i386
does not have a move that would also decrement one of its operands before move.


-- 

ubizjak at gmail dot com changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |ubizjak at gmail dot com


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=31344

[Bug bootstrap/31344] [4.3 Regression] bootstrap broken on i[345]86-linux

[ubizjak at gmail dot com]

------- Comment #20 from ubizjak at gmail dot com  2007-05-07 08:51 -------
Following one-liner fixes the failure. Note that this is for i386 only, as we
also need to skip other autoinc/autodec references.

This is now a generic RTL problem.

2007-05-07  Uros Bizjak  <ubizjak@gmail.com>

        * explow.c (memory_address): Do not force autoincremented or
        autodecremented address references into pseudo register.

Index: explow.c
===================================================================
--- explow.c    (revision 124185)
+++ explow.c    (working copy)
@@ -445,6 +445,10 @@
         transformations can make better code.  */
       LEGITIMIZE_ADDRESS (x, oldx, mode, win);

+      /* Skip autoincremented/autodecremented references.  */
+      if (GET_CODE (x) == PRE_DEC)
+       goto done;
+
       /* PLUS and MULT can appear in special ways
         as the result of attempts to make an address usable for indexing.
         Usually they are dealt with by calling force_operand, below.


-- 


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=31344

[Bug rtl-optimization/31344] [4.3 Regression] bootstrap broken on i[345]86-linux

[ubizjak at gmail dot com]

------- Comment #21 from ubizjak at gmail dot com  2007-05-07 11:11 -------
Patch at http://gcc.gnu.org/ml/gcc-patches/2007-05/msg00390.html


-- 

ubizjak at gmail dot com changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
         AssignedTo|unassigned at gcc dot gnu   |ubizjak at gmail dot com
                   |dot org                     |
                URL|                            |http://gcc.gnu.org/ml/gcc-
                   |                            |patches/2007-
                   |                            |05/msg00390.html
           Severity|blocker                     |major
             Status|NEW                         |ASSIGNED
          Component|bootstrap                   |rtl-optimization
           Keywords|                            |patch
   Last reconfirmed|2007-03-31 16:01:43         |2007-05-07 11:11:16
               date|                            |


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=31344

[Bug rtl-optimization/31344] [4.3 Regression] bootstrap broken on i[345]86-linux

[ubizjak at gmail dot com]

------- Comment #22 from ubizjak at gmail dot com  2007-05-11 14:09 -------
Alternative patch to emit_move_change_mode() to take push_operand away from
change_address():

Index: expr.c
===================================================================
--- expr.c      (revision 124612)
+++ expr.c      (working copy)
@@ -2867,7 +2867,12 @@ emit_move_change_mode (enum machine_mode
 {
   rtx ret;

-  if (MEM_P (x))
+  if (push_operand (x, GET_MODE (x)))
+    {
+      ret = gen_rtx_MEM (new_mode, XEXP (x, 0));
+      MEM_COPY_ATTRIBUTES (ret, x);
+    }
+  else if (MEM_P (x))
     {
       /* We don't have to worry about changing the address since the
         size in bytes is supposed to be the same.  */


-- 


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=31344

[Bug rtl-optimization/31344] [4.3 Regression] bootstrap broken on i[345]86-linux

[uros at gcc dot gnu dot org]

------- Comment #23 from uros at gcc dot gnu dot org  2007-05-18 09:37 -------
Subject: Bug 31344

Author: uros
Date: Fri May 18 08:37:03 2007
New Revision: 124825

URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=124825
Log:
        PR rtl-optimization/31344
        * expr.c (emit_move_change_mode): Change mode of push operands here.

testsuite/ChangeLog:

        PR rtl-optimization/31344
        * gcc.dg/pr31344.c: New test.


Added:
    trunk/gcc/testsuite/gcc.dg/pr31344.c
Modified:
    trunk/gcc/ChangeLog
    trunk/gcc/expr.c
    trunk/gcc/testsuite/ChangeLog


-- 


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=31344

[Bug rtl-optimization/31344] [4.3 Regression] bootstrap broken on i[345]86-linux

[uros at gcc dot gnu dot org]

------- Comment #24 from uros at gcc dot gnu dot org  2007-05-18 09:46 -------
Subject: Bug 31344

Author: uros
Date: Fri May 18 08:46:30 2007
New Revision: 124826

URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=124826
Log:
        * PR rtl-optimization/31344 is actually middle-end bug.

Modified:
    trunk/gcc/ChangeLog
    trunk/gcc/testsuite/ChangeLog


-- 


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=31344

[Bug middle-end/31344] [4.3 Regression] bootstrap broken on i[345]86-linux

[ubizjak at gmail dot com]

------- Comment #25 from ubizjak at gmail dot com  2007-05-18 09:48 -------
Fixed.


-- 

ubizjak at gmail dot com changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                URL|http://gcc.gnu.org/ml/gcc-  |http://gcc.gnu.org/ml/gcc-
                   |patches/2007-               |patches/2007-
                   |05/msg00390.html            |05/msg01192.html
             Status|ASSIGNED                    |RESOLVED
          Component|rtl-optimization            |middle-end
         Resolution|                            |FIXED


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=31344