Fix intermittent failures on the H8, particularly H8/SX tests.
authorJeff Law <jeffreyalaw@gmail.com>
Sat, 20 Nov 2021 18:06:15 +0000 (13:06 -0500)
committerJeff Law <jeffreyalaw@gmail.com>
Sat, 20 Nov 2021 18:06:15 +0000 (13:06 -0500)
commitdbf98db6f073cc646b20b5dcfabfadeec61b4ea3
treecc62c4f639bcad7f76248e9ecb534b6a178c7c9f
parent911438f9f4516f2c5c3fc4eaecc47571aef98d1d
Fix intermittent failures on the H8, particularly H8/SX tests.

    The upstream GCC tester has  showed spurious execution failures on the
    H8 target for the H8/SX multilibs. I suspected memory corruption or an
    uninitialized variable early as the same binary would sometimes work and
    sometimes it got the wrong result. Worse yet, the point where the test
    determined it was getting the wrong result would change.

    Because it only happened on the H8/SX variant I was able to zero in on
    the "mova" support and the "short form" of those instructions in particular.

    As the code stands it checks if code->op3.type == 0 to try and identify cases
    where op3 wasn't filled in and thus we've got the short form of the mova
    instruction.

    But for the short-form of those instructions we never set any of the "op3"
    data structure. We get whatever was lying around -- it's usually zero and
     thus things usually work, but if the stale data was nonzero, then we'd
    fail to recognize the instruction as a short-form and fail to set up the
    various fields appropriately.

    I initially initialized the op3.type field to zero, but didn't like that
     because it was inconsistent with how other operands were initialized.
    Bringing consistency meant using -1 as the initializer value and adjusting
    the check for short form mova appropriately.

    I've had this in the upstream GCC tester for perhaps a year at this point
    and haven't seen any of the intermittent failures again.
sim/h8300/compile.c