tree 15487f86c06dd3a2e563030cb5c1ee68f41f0684
parent c013bc85f45add8c4bc8c7163316bbb90b37a15e
author Matthew Bentham <matthew.bentham@arm.com> 1625496325 +0100
committer Sadik Armagan <sadik.armagan@arm.com> 1626446349 +0000

Avoid undefined behaviour in NumericCast unit test

Because the maximum value of an int32_t isn't exactly representable
by a 32-bit float, casting it to a float produces a greater value,
and then casting that back to an int32_t is undefined. (In tested
implementations the cast saturates, so accidentally the correct
result is returned).

Using double for the intermediate value instead gives a high
enough precision that this test case can be safely expressed.

Silences warning from Undefioned Behaviour Sanitiser.

Signed-off-by: Matthew Bentham <matthew.bentham@arm.com>
Change-Id: I6fe3e89cb6ac695049c92f3b75aca0c14d0e14aa