Fix "bins" simulation for v850e3v5
authorJeff Law <jeffreyalaw@gmail.com>
Wed, 6 Apr 2022 15:06:53 +0000 (11:06 -0400)
committerJeff Law <jeffreyalaw@gmail.com>
Wed, 6 Apr 2022 15:06:53 +0000 (11:06 -0400)
commit49fffa58f7e6da777d10fe77663bc7c8f531fe7f
tree9b2efafe99b733a62bffb45b61c965a7a80979cc
parent7fb56b98937a2feef5a3e12d8b00506ff4d132be
Fix "bins" simulation for v850e3v5

I've been carrying this for a few years.   One test in the GCC testsuite is
failing due to a bug in the handling of the v850e3v5 instruction "bins".

When the "bins" instruction specifies a 32bit bitfield size, the simulator
exhibits undefined behavior by trying to shift a 32 bit quantity by 32 bits.
In the case of a 32 bit shift, we know what the resultant mask should be.  So
we can just set it.

That seemed better than using 1UL for the constant (on a 32bit host unsigned
long might still just be 32 bits) or needlessly forcing everything to
long long types.

Thankfully the case where this shows up is only bins <src>, 0, 32, <dest>
which would normally be encoded as a simple move.

* testsuite/v850/allinsns.exp: Add v850e3v5.
* testsuite/v850/bins.cgs: New test.
* v850/simops.c (v850_bins): Avoid undefined behavior on left shift.
sim/testsuite/v850/allinsns.exp
sim/testsuite/v850/bins.cgs [new file with mode: 0644]
sim/v850/simops.c