stillwater-sc / universal / 30642798036

85%

master: 84%

push

github

fix(native): restore inline exponent mask in setFields (-Wconversion regression) (#1265) (#1278)

#1262 extracted `const uint64_t exponent = rawExponentBits & 0x7FFF;` into a local
and assigned it to the 15-bit `parts.exponent` bitfield. That broke gcc's range
tracking: the inline `& 0x7FFF` proved the value fits :15 (no warning), but a
uint64_t local can hold values that do not, so `parts.exponent = exponent` became
a narrowing (-Wconversion) -- re-introducing exactly the kind of warning the
warning-clean epic (#1265) exists to remove (set_fields.hpp:100).

Inline the mask at the exponent assignment again, and derive bit63 from the same
masked expression. Behavior is unchanged (the value is identical); this only
restores the compiler's ability to see the truncation is safe.

With this, all 8 headers in the #1265 inventory are clean under gcc
-Wconversion / -Wfloat-conversion. gcc + clang: native_float_ieee754 PASS
(long double field round-trip unchanged).

Relates to #1262, #1265

2 of 2 new or added lines in 1 file covered. (100.0%)

41697 of 48912 relevant lines covered (85.25%)

7267666.96 hits per line