[Bug target/16719] New: [ColdFire] Illegal move of byte itno address register causes compiler to ICE

[Bug target/16719] New: [ColdFire] Illegal move of byte itno address register causes compiler to ICE

[peter at the-baradas dot com]

I've stumbled across yet another problem with trying to move a byte into an
address registe ron ColdFire.  This problem was pointed out to me by Brett
Swimley, and was reproducedd using today's version of uberbaum.

[peter@baradas peter]$ /tmp/uberbaum/bin/m68k-elf-gcc -I./ -Wall -Wsign-compare
-W -Wno-unused -Wshadow -m5206e -O3 -fomit-frame-pointer -funroll-loops   -S -o
/tmp/safer.s ~/tmp/safer.c
/home/peter/tmp/safer.c: In function `safer_ecb_encrypt':
/home/peter/tmp/safer.c:2494: error: insn does not satisfy its constraints:
(insn 319 318 320 10 (set (reg:QI 14 %a6)
        (mem/s:QI (plus:SI (reg:SI 14 %a6)
                (reg:SI 0 %d0 [104])) [0 safer_ebox S1 A8])) 34 {*m68k.md:752} (nil)
    (nil))
/home/peter/tmp/safer.c:2494: internal compiler error: in
reload_cse_simplify_operands, at postreload.c:378
Please submit a full bug report,
with preprocessed source if appropriate.
See <URL:http://gcc.gnu.org/bugs.html> for instructions.


Source code the reproduces the bug:



typedef unsigned char safer_key_t[(1 + 8 * (1 + 2 * 13))];
struct safer_key { safer_key_t key; };


typedef union Symmetric_key {

   struct safer_key safer;

} symmetric_key;

extern void crypt_argchk(char *v, char *s, int d);
extern const unsigned char safer_ebox[], safer_lbox[];

void safer_ecb_encrypt(const unsigned char *block_in,
                             unsigned char *block_out,
                             symmetric_key *skey)

{

    unsigned char a, b, c, d, e, f, g, h, t;
    unsigned int round;
    unsigned char *key;

    if (!(block_in != ((void *)0))) { crypt_argchk("block_in != NULL",
"safer.c", 248); };
    if (!(block_out != ((void *)0))) { crypt_argchk("block_out != NULL",
"safer.c", 249); };
    if (!(skey != ((void *)0))) { crypt_argchk("skey != NULL", "safer.c", 250); };

    key = skey->safer.key;
    a = block_in[0]; b = block_in[1]; c = block_in[2]; d = block_in[3];
    e = block_in[4]; f = block_in[5]; g = block_in[6]; h = block_in[7];
    if (13 < (round = *key)) round = 13;
    while(round-- > 0)
    {
        a ^= *++key; b += *++key; c += *++key; d ^= *++key;
        e ^= *++key; f += *++key; g += *++key; h ^= *++key;
        a = safer_ebox[(a) & 0xFF] + *++key; b = safer_lbox[(b) & 0xFF] ^ *++key;
        c = safer_lbox[(c) & 0xFF] ^ *++key; d = safer_ebox[(d) & 0xFF] + *++key;
        e = safer_ebox[(e) & 0xFF] + *++key; f = safer_lbox[(f) & 0xFF] ^ *++key;
        g = safer_lbox[(g) & 0xFF] ^ *++key; h = safer_ebox[(h) & 0xFF] + *++key;
        { b += a; a += b; }; { d += c; c += d; }; { f += e; e += f; }; { h += g;
g += h; };
        { c += a; a += c; }; { g += e; e += g; }; { d += b; b += d; }; { h += f;
f += h; };
        { e += a; a += e; }; { f += b; b += f; }; { g += c; c += g; }; { h += d;
d += h; };
        t = b; b = e; e = c; c = t; t = d; d = f; f = g; g = t;
    }
    a ^= *++key; b += *++key; c += *++key; d ^= *++key;
    e ^= *++key; f += *++key; g += *++key; h ^= *++key;
    block_out[0] = a & 0xFF; block_out[1] = b & 0xFF;
    block_out[2] = c & 0xFF; block_out[3] = d & 0xFF;
    block_out[4] = e & 0xFF; block_out[5] = f & 0xFF;
    block_out[6] = g & 0xFF; block_out[7] = h & 0xFF;
}

-- 
           Summary: [ColdFire] Illegal move of byte itno address register
                    causes compiler to ICE
           Product: gcc
           Version: 3.5.0
            Status: UNCONFIRMED
          Severity: normal
          Priority: P2
         Component: target
        AssignedTo: unassigned at gcc dot gnu dot org
        ReportedBy: peter at the-baradas dot com
                CC: gcc-bugs at gcc dot gnu dot org,peter at the-baradas dot
                    com
GCC target triplet: m68k-elf


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

[Bug target/16719] [ColdFire] Illegal move of byte itno address register causes compiler to ICE

[peter at the-baradas dot com]

------- Additional Comments From peter at the-baradas dot com  2004-07-26 17:26 -------
Created an attachment (id=6828)
 --> (http://gcc.gnu.org/bugzilla/attachment.cgi?id=6828&action=view)
Testcase to reproduce the bug


-- 


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

[Bug target/16719] [ColdFire] Illegal move of byte itno address register causes compiler to ICE

[pinskia at gcc dot gnu dot org]

-- 
           What    |Removed                     |Added
----------------------------------------------------------------------------
   Attachment #6828|application/octet-stream    |text/plain
          mime type|                            |


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

[Bug target/16719] [ColdFire] Illegal move of byte itno address register causes compiler to ICE

[pinskia at gcc dot gnu dot org]

------- Additional Comments From pinskia at gcc dot gnu dot org  2004-10-25 00:59 -------
Confirmed, here is a reduced testcase:
unsigned char a, b, c, d, e, f, g;
void safer_ecb_encrypt(unsigned char *block_out, unsigned char *key, unsigned int round)
{
    while(round-- > 0)
    {
        a ^= *++key; b += *++key; c += *++key; d ^= *++key;
        e ^= *++key; f += *++key; g += *++key; 
    }
    block_out[0] = a & 0xFF; block_out[1] = b & 0xFF;
    block_out[2] = c & 0xFF; block_out[3] = d & 0xFF;
    block_out[4] = e & 0xFF; block_out[5] = f & 0xFF;
    block_out[6] = g & 0xFF;
}

-- 
           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|UNCONFIRMED                 |NEW
     Ever Confirmed|                            |1
   Last reconfirmed|0000-00-00 00:00:00         |2004-10-25 00:59:01
               date|                            |


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

[Bug target/16719] [ColdFire] Illegal move of byte itno address register causes compiler to ICE

[ralf dot corsepius at rtems dot org]

------- Additional Comments From ralf dot corsepius at rtems dot org  2005-01-27 17:31 -------
(In reply to comment #2)
> Confirmed, here is a reduced testcase:
I can't confirm this ICE with your reduced testcase and m68k-rtems-gcc.
But I can confirm m68k-rtems-gcc to be ICEing with the original example (safer.c).

# m68k-rtems4.7-gcc --version
m68k-rtems4.7-gcc (GCC) 4.0.0 20050126 (experimental)

/tmp/safer.c: In function 'safer_ecb_encrypt':
/tmp/safer.c:2494: error: insn does not satisfy its constraints:
(insn 301 300 302 10 (set (reg:QI 9 %a1)
        (mem/s/u:QI (plus:SI (reg:SI 9 %a1)
                (reg:SI 0 %d0 [91])) [0 safer_ebox S1 A8])) 34 {*m68k.md:752} (nil)
    (nil))
/tmp/safer.c:2494: internal compiler error: in reload_cse_simplify_operands, at
postreload.c:391


-- 


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

[Bug target/16719] [ColdFire] Illegal move of byte itno address register causes compiler to ICE

[peter at the-baradas dot com]

------- Additional Comments From peter at the-baradas dot com  2005-01-27 18:15 -------
See the following for a patch waiting for approval...

http://gcc.gnu.org/ml/gcc-patches/2004-12/msg00405.html

-- 


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

[Bug target/16719] [ColdFire] Illegal move of byte itno address register causes compiler to ICE

[ralf dot corsepius at rtems dot org]

------- Additional Comments From ralf dot corsepius at rtems dot org  2005-01-28 14:11 -------
The patch contained in
http://gcc.gnu.org/ml/gcc-patches/2004-12/msg00405.html
doesn't seem to work for me, rsp. triggers another ICE, at least the error
message seems to have changed:

With today's gcc/cvs-trunk + your patch applied:

# m68k-rtems4.7-gcc -W -Wno-unused -Wshadow -m5206e -O3 -fomit-frame-pointer
-funroll-loops -S -o tmp.s /tmp/safer.c
/tmp/safer.c: In function 'safer_ecb_encrypt':
/tmp/safer.c:2494: error: unable to find a register to spill in class 'ADDR_REGS'
/tmp/safer.c:2494: error: this is the insn:
(insn 95 94 97 10 (set (reg:SI 8 %a0 [95])
        (plus:SI (subreg:SI (mem/s/u:QI (plus:SI (reg/f:SI 13 %a5 [189])
                        (reg:SI 0 %d0 [91])) [0 safer_ebox S1 A8]) 0)
            (subreg:SI (mem:QI (plus:SI (reg/v/f:SI 10 %a2 [orig:39 key.78 ] [39])
                        (const_int 9 [0x9])) [0 S1 A8]) 0))) 96 {*addsi3_5200}
(insn_list:REG_DEP_TRUE 91 (nil))
    (expr_list:REG_DEAD (reg:SI 0 %d0 [91])
        (nil)))
/tmp/safer.c:2494: internal compiler error: in spill_failure, at reload1.c:1872

Also I am seeing a new ICE during building RTEMS, but I am not sure if this is
related to your patch.

-- 
           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |joel at oarcorp dot com


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

[Bug target/16719] [ColdFire] Illegal move of byte itno address register causes compiler to ICE

[cvs-commit at gcc dot gnu dot org]

------- Additional Comments From cvs-commit at gcc dot gnu dot org  2005-07-11 23:32 -------
Subject: Bug 16719

CVSROOT:	/cvs/gcc
Module name:	gcc
Changes by:	bernie@gcc.gnu.org	2005-07-11 23:32:01

Modified files:
	gcc            : ChangeLog 
	gcc/config/m68k: m68k.md m68k.c m68k.h m68k-protos.h 

Log message:
	PR middle-end/16719
	PR middle-end/18421
	* config/m68k/m68k.h (HARD_REGNO_MODE_OK): Disallow bytes
	in address registers.
	* config/m68k/m68k.c (hard_regno_mode_ok): Likewise.
	* config/m68k/m68k.md: Replace 's' with 'i' in 4th
	alternative of addsi3_5200.

Patches:
http://gcc.gnu.org/cgi-bin/cvsweb.cgi/gcc/gcc/ChangeLog.diff?cvsroot=gcc&r1=2.9411&r2=2.9412
http://gcc.gnu.org/cgi-bin/cvsweb.cgi/gcc/gcc/config/m68k/m68k.md.diff?cvsroot=gcc&r1=1.84&r2=1.85
http://gcc.gnu.org/cgi-bin/cvsweb.cgi/gcc/gcc/config/m68k/m68k.c.diff?cvsroot=gcc&r1=1.152&r2=1.153
http://gcc.gnu.org/cgi-bin/cvsweb.cgi/gcc/gcc/config/m68k/m68k.h.diff?cvsroot=gcc&r1=1.128&r2=1.129
http://gcc.gnu.org/cgi-bin/cvsweb.cgi/gcc/gcc/config/m68k/m68k-protos.h.diff?cvsroot=gcc&r1=1.20&r2=1.21



-- 


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

[Bug target/16719] [ColdFire] Illegal move of byte itno address register causes compiler to ICE

[bernie at develer dot com]

------- Additional Comments From bernie at develer dot com  2005-07-11 23:25 -------
Patch attached to Bug #18421 (Attachment #9246).


-- 


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

[Bug target/16719] [ColdFire] Illegal move of byte itno address register causes compiler to ICE

[bernie at develer dot com]

------- Additional Comments From bernie at develer dot com  2005-07-11 23:34 -------
Fixed on mainline.  Pending for 4.0 and 3.4.

-- 


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

[Bug target/16719] [ColdFire] Illegal move of byte itno address register causes compiler to ICE

[cvs-commit at gcc dot gnu dot org]

------- Additional Comments From cvs-commit at gcc dot gnu dot org  2005-07-26 04:40 -------
Subject: Bug 16719

CVSROOT:	/cvs/gcc
Module name:	gcc
Branch: 	gcc-4_0-branch
Changes by:	bernie@gcc.gnu.org	2005-07-26 04:40:44

Modified files:
	gcc            : ChangeLog 
	gcc/config/m68k: m68k.md m68k.h m68k.c m68k-protos.h 

Log message:
	Backport from mainline:
	2005-07-12  Peter Barada  <peter@the-baradas.com>
	PR middle-end/16719
	PR middle-end/18421
	* config/m68k/m68k.h (HARD_REGNO_MODE_OK): Disallow bytes
	in address registers.
	* config/m68k/m68k.c (hard_regno_mode_ok): Likewise.
	* config/m68k/m68k.md: Replace 's' with 'i' in 4th
	alternative of addsi3_5200.

Patches:
http://gcc.gnu.org/cgi-bin/cvsweb.cgi/gcc/gcc/ChangeLog.diff?cvsroot=gcc&only_with_tag=gcc-4_0-branch&r1=2.7592.2.331&r2=2.7592.2.332
http://gcc.gnu.org/cgi-bin/cvsweb.cgi/gcc/gcc/config/m68k/m68k.md.diff?cvsroot=gcc&only_with_tag=gcc-4_0-branch&r1=1.80&r2=1.80.34.1
http://gcc.gnu.org/cgi-bin/cvsweb.cgi/gcc/gcc/config/m68k/m68k.h.diff?cvsroot=gcc&only_with_tag=gcc-4_0-branch&r1=1.123&r2=1.123.10.1
http://gcc.gnu.org/cgi-bin/cvsweb.cgi/gcc/gcc/config/m68k/m68k.c.diff?cvsroot=gcc&only_with_tag=gcc-4_0-branch&r1=1.144&r2=1.144.2.1
http://gcc.gnu.org/cgi-bin/cvsweb.cgi/gcc/gcc/config/m68k/m68k-protos.h.diff?cvsroot=gcc&only_with_tag=gcc-4_0-branch&r1=1.17&r2=1.17.18.1



-- 


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

[Bug target/16719] [ColdFire] Illegal move of byte itno address register causes compiler to ICE

[cvs-commit at gcc dot gnu dot org]

------- Additional Comments From cvs-commit at gcc dot gnu dot org  2005-07-26 20:32 -------
Subject: Bug 16719

CVSROOT:	/cvs/gcc
Module name:	gcc
Branch: 	gcc-3_4-branch
Changes by:	bernie@gcc.gnu.org	2005-07-26 20:32:26

Modified files:
	gcc            : ChangeLog 
	gcc/config/m68k: m68k-protos.h m68k.c m68k.h m68k.md 

Log message:
	Backport from mainline:
	2005-07-12  Peter Barada  <peter@the-baradas.com>
	PR middle-end/16719
	PR middle-end/18421
	* config/m68k/m68k.h (HARD_REGNO_MODE_OK): Disallow bytes
	in address registers.
	* config/m68k/m68k.c (hard_regno_mode_ok): Likewise.
	* config/m68k/m68k.md: Replace 's' with 'i' in 4th
	alternative of addsi3_5200.

Patches:
http://gcc.gnu.org/cgi-bin/cvsweb.cgi/gcc/gcc/ChangeLog.diff?cvsroot=gcc&only_with_tag=gcc-3_4-branch&r1=2.2326.2.884&r2=2.2326.2.885
http://gcc.gnu.org/cgi-bin/cvsweb.cgi/gcc/gcc/config/m68k/m68k-protos.h.diff?cvsroot=gcc&only_with_tag=gcc-3_4-branch&r1=1.14.10.1&r2=1.14.10.2
http://gcc.gnu.org/cgi-bin/cvsweb.cgi/gcc/gcc/config/m68k/m68k.c.diff?cvsroot=gcc&only_with_tag=gcc-3_4-branch&r1=1.123.2.3&r2=1.123.2.4
http://gcc.gnu.org/cgi-bin/cvsweb.cgi/gcc/gcc/config/m68k/m68k.h.diff?cvsroot=gcc&only_with_tag=gcc-3_4-branch&r1=1.103.4.5&r2=1.103.4.6
http://gcc.gnu.org/cgi-bin/cvsweb.cgi/gcc/gcc/config/m68k/m68k.md.diff?cvsroot=gcc&only_with_tag=gcc-3_4-branch&r1=1.71.4.3&r2=1.71.4.4



-- 


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

[Bug target/16719] [ColdFire] Illegal move of byte itno address register causes compiler to ICE

[pinskia at gcc dot gnu dot org]

------- Additional Comments From pinskia at gcc dot gnu dot org  2005-07-26 20:34 -------
Fixed.

-- 
           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|NEW                         |RESOLVED
      Known to fail|                            |4.0.0 3.4.4
      Known to work|                            |3.4.5 4.0.2 4.1.0
         Resolution|                            |FIXED
   Target Milestone|---                         |3.4.5


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