diff options
| -rw-r--r-- | gcc/ChangeLog | 7 | ||||
| -rw-r--r-- | gcc/config/m32c/m32c.c | 39 |
2 files changed, 27 insertions, 19 deletions
diff --git a/gcc/ChangeLog b/gcc/ChangeLog index 9cb17ffa376..59ad7fc3ded 100644 --- a/gcc/ChangeLog +++ b/gcc/ChangeLog @@ -1,3 +1,10 @@ +2007-02-08 DJ Delorie <dj@redhat.com> + + * config/m32c/m32c.c (m32c_unpend_compare): Add default to silence + warnings. + (legal_subregs): Use unsigned char, make const. + (m32c_illegal_subreg_p): Use ARRAY_SIZE. Delete unused variables. + 2007-02-08 Paul Brook <paul@codesourcery.com> * config/arm/lib1funcs.asm (RETLDM): Pop directly into PC when no diff --git a/gcc/config/m32c/m32c.c b/gcc/config/m32c/m32c.c index 65f0898ee1b..de7ce9d7a0f 100644 --- a/gcc/config/m32c/m32c.c +++ b/gcc/config/m32c/m32c.c @@ -2750,28 +2750,28 @@ m32c_insert_attributes (tree node ATTRIBUTE_UNUSED, /* Predicates */ /* This is a list of legal subregs of hard regs. */ -static struct { - enum machine_mode outer_mode_size; - enum machine_mode inner_mode_size; +static const struct { + unsigned char outer_mode_size; + unsigned char inner_mode_size; + unsigned char byte_mask; + unsigned char legal_when; unsigned int regno; - int byte_mask; - int legal_when; } legal_subregs[] = { - {1, 2, R0_REGNO, 0x03, 1}, /* r0h r0l */ - {1, 2, R1_REGNO, 0x03, 1}, /* r1h r1l */ - {1, 2, A0_REGNO, 0x01, 1}, - {1, 2, A1_REGNO, 0x01, 1}, + {1, 2, 0x03, 1, R0_REGNO}, /* r0h r0l */ + {1, 2, 0x03, 1, R1_REGNO}, /* r1h r1l */ + {1, 2, 0x01, 1, A0_REGNO}, + {1, 2, 0x01, 1, A1_REGNO}, - {1, 4, A0_REGNO, 0x01, 1}, - {1, 4, A1_REGNO, 0x01, 1}, + {1, 4, 0x01, 1, A0_REGNO}, + {1, 4, 0x01, 1, A1_REGNO}, - {2, 4, R0_REGNO, 0x05, 1}, /* r2 r0 */ - {2, 4, R1_REGNO, 0x05, 1}, /* r3 r1 */ - {2, 4, A0_REGNO, 0x05, 16}, /* a1 a0 */ - {2, 4, A0_REGNO, 0x01, 24}, /* a1 a0 */ - {2, 4, A1_REGNO, 0x01, 24}, /* a1 a0 */ + {2, 4, 0x05, 1, R0_REGNO}, /* r2 r0 */ + {2, 4, 0x05, 1, R1_REGNO}, /* r3 r1 */ + {2, 4, 0x05, 16, A0_REGNO}, /* a1 a0 */ + {2, 4, 0x01, 24, A0_REGNO}, /* a1 a0 */ + {2, 4, 0x01, 24, A1_REGNO}, /* a1 a0 */ - {4, 8, R0_REGNO, 0x55, 1}, /* r3 r1 r2 r0 */ + {4, 8, 0x55, 1, R0_REGNO}, /* r3 r1 r2 r0 */ }; /* Returns TRUE if OP is a subreg of a hard reg which we don't @@ -2779,7 +2779,6 @@ static struct { bool m32c_illegal_subreg_p (rtx op) { - rtx orig_op = op; int offset; unsigned int i; int src_mode, dest_mode; @@ -2801,7 +2800,7 @@ m32c_illegal_subreg_p (rtx op) offset = (1 << offset); - for (i = 0; i < sizeof(legal_subregs)/sizeof(legal_subregs[0]); i ++) + for (i = 0; i < ARRAY_SIZE (legal_subregs); i ++) if (legal_subregs[i].outer_mode_size == GET_MODE_SIZE (dest_mode) && legal_subregs[i].regno == REGNO (op) && legal_subregs[i].inner_mode_size == GET_MODE_SIZE (src_mode) @@ -3584,6 +3583,8 @@ m32c_unpend_compare (void) emit_insn (gen_cmphi_op (compare_op0, compare_op1)); case PSImode: emit_insn (gen_cmppsi_op (compare_op0, compare_op1)); + default: + /* Just to silence the "missing case" warnings. */ ; } } |