mips: Clean up type overrides for operands.
authorGabe Black <gabeblack@google.com>
Sat, 13 Oct 2018 06:05:40 +0000 (23:05 -0700)
committerGabe Black <gabeblack@google.com>
Tue, 6 Nov 2018 01:06:05 +0000 (01:06 +0000)
commit223840f0b4b99036f8706f531d20a2ccc8fd8bf4
tree724fd32ce827917773c0e1e48a2aab526d0a5d2a
parent0167f66c130ada7a49d59d1ef4644d282cd146dc
mips: Clean up type overrides for operands.

For operands which default to uw (uint32_t), there's no reason to
explicitly specify that all over the place. Also, when assigning to a
32 bit value which is supposed to be the full width of the resulting
register, there's no reason to override the value to be signed. If the
value is expanded into a larger value, then extra bits may get set
unintentionally through sign extension. Even if an instruction
produces a value which should be interpreted as signed, it will still
only produce a value of a certain predefined width, even if that answer
ends up stored in a larger variable.

Change-Id: I048d68c5dd08a1d40e8117ae9d36d70e05ec21c8
Reviewed-on: https://gem5-review.googlesource.com/c/13618
Reviewed-by: Jason Lowe-Power <jason@lowepower.com>
Maintainer: Gabe Black <gabeblack@google.com>
src/arch/mips/isa/decoder.isa