Fix bug in fp16_alt_to_fp32_value

Wrong type for exp_offset constant didn't cause correctness issues as the constant fit into the wrong type, but potentially caused poor code-generation
Maratyszcza · Apr 12, 2020 · 3c54eac · 3c54eac
1 parent ba1d31f
commit 3c54eac
Showing 1 changed file with 1 addition and 1 deletion.
diff --git a/include/fp16/fp16.h b/include/fp16/fp16.h
@@ -373,7 +373,7 @@ static inline float fp16_alt_to_fp32_value(uint16_t h) {
 	 * Note that this operation does not handle denormal inputs (where biased exponent == 0). However, they also do not
 	 * operate on denormal inputs, and do not produce denormal results.
 	 */
-	const float exp_offset = UINT32_C(0x70) << 23;
+	const uint32_t exp_offset = UINT32_C(0x70) << 23;
 	const float normalized_value = fp32_from_bits((two_w >> 4) + exp_offset);
 
 	/*