I was not aware this incurred undefined behaviour; thank you cppcheck.
Signed-off-by: Alyssa Rosenzweig <alyssa.rosenzweig@collabora.com>
Reviewed-by: Tomeu Vizoso <tomeu.vizoso@collabora.com>
unsigned m = m_f;
/* Default case */
- unsigned magic_divisor = m, extra_flags = 0;
+ uint32_t magic_divisor = m, extra_flags = 0;
/* e = 2^(shift + 32) % d */
uint64_t e = t % hw_divisor;
}
/* Top flag implicitly set */
- assert(magic_divisor & (1 << 31));
- magic_divisor &= ~(1 << 31);
+ assert(magic_divisor & (1u << 31));
+ magic_divisor &= ~(1u << 31);
/* Upload to two different slots */