[matt-edmondson]

Fixes #376

What changed

GoGenerator.WriteFloatingRemainder wrote both operands of a double remainder unchanged. math.Mod takes only float64, and Go converts nothing implicitly, so any int or float32 operand broke the build. The double branch now passes each operand through GenerateWidenedTo(operand, double), which writes float64(...) only around an operand narrower than float64. The float32 branch already converted both operands.

double rem(float x, double d, int i) { double r = x % d; r %= i; return r + (i % d); } now generates:

var r float64 = math.Mod(float64(x), d)
r = math.Mod(r, float64(i))
return (r + math.Mod(float64(i), d))

Tests

• New Go_MixedOperandsOfADoubleRemainderAreWidened checks the spelling of all three remainders, including the compound %=.
• RemaindersCompile now also compiles the mixed-operand file for C, C++ and Go (go vet).
• With the fix reverted, both tests fail. go vet reports cannot use x (variable of type float32) as float64 value in argument to math.Mod.
• Full suite: 1665 passed, 0 failed.

---

Generated by Claude Code

[sonarqubecloud]

Quality Gate passed

Issues
3 New issues
0 Accepted issues

Measures
0 Security Hotspots
100.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud