Skip to content

Commit

Permalink
math_brute_force: always initialize oldMode (#1796)
Browse files Browse the repository at this point in the history
Avoid a maybe-uninitialized warning by ensuring that `oldMode` is
always initialized to 0.  There is no need to use `memset` for this,
as `FPU_mode_type` is either an `int` or an `int64_t`.

Signed-off-by: Sven van Haastregt <sven.vanhaastregt@arm.com>
  • Loading branch information
svenvh authored Aug 7, 2023
1 parent aa23f34 commit ddf46ca
Showing 1 changed file with 1 addition and 2 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -189,12 +189,11 @@ int TestFunc_Float2_Float(const Func *f, MTdata d, bool relaxedMode)
// Get that moving
if ((error = clFlush(gQueue))) vlog("clFlush failed\n");

FPU_mode_type oldMode;
FPU_mode_type oldMode = 0;
RoundingMode oldRoundMode = kRoundToNearestEven;
if (isFract)
{
// Calculate the correctly rounded reference result
memset(&oldMode, 0, sizeof(oldMode));
if (ftz || relaxedMode) ForceFTZ(&oldMode);

// Set the rounding mode to match the device
Expand Down

0 comments on commit ddf46ca

Please sign in to comment.