diff options
author | Nick Clifton <nickc@redhat.com> | 2006-03-05 08:38:53 +0000 |
---|---|---|
committer | Nick Clifton <nickc@redhat.com> | 2006-03-05 08:38:53 +0000 |
commit | ed963e2de8fb9d28b2540e2bc23ad19b3670ebea (patch) | |
tree | 6aecf5a6023f9cbe35a0ba771724965aea3640dc /opcodes/m32c-ibld.c | |
parent | 3c568553f111f6a986a5a9fbd882fcd09ae4656a (diff) | |
download | binutils-ed963e2de8fb9d28b2540e2bc23ad19b3670ebea.zip binutils-ed963e2de8fb9d28b2540e2bc23ad19b3670ebea.tar.gz binutils-ed963e2de8fb9d28b2540e2bc23ad19b3670ebea.tar.bz2 |
* cgen-ibld.in (insert_normal): Cope with attempts to insert a signed 32-bit
value into an unsigned 32-bit field when the host is a 64-bit machine.
Diffstat (limited to 'opcodes/m32c-ibld.c')
-rw-r--r-- | opcodes/m32c-ibld.c | 16 |
1 files changed, 12 insertions, 4 deletions
diff --git a/opcodes/m32c-ibld.c b/opcodes/m32c-ibld.c index 62c753b..7601128 100644 --- a/opcodes/m32c-ibld.c +++ b/opcodes/m32c-ibld.c @@ -168,13 +168,21 @@ insert_normal (CGEN_CPU_DESC cd, else if (! CGEN_BOOL_ATTR (attrs, CGEN_IFLD_SIGNED)) { unsigned long maxval = mask; - - if ((unsigned long) value > maxval) + unsigned long val = (unsigned long) value; + + /* For hosts with a word size > 32 check to see if value has been sign + extended beyond 32 bits. If so then ignore these higher sign bits + as the user is attempting to store a 32-bit signed value into an + unsigned 32-bit field which is allowed. */ + if (sizeof (unsigned long) > 4 && ((value >> 32) == -1)) + val &= 0xFFFFFFFF; + + if (val > maxval) { /* xgettext:c-format */ sprintf (errbuf, - _("operand out of range (%lu not between 0 and %lu)"), - value, maxval); + _("operand out of range (0x%lx not between 0 and 0x%lx)"), + val, maxval); return errbuf; } } |