cmd/compile: miscompilation of float64 comparison and bool comparison #34587
Labels
FrozenDueToAge
WaitingForInfo
Issue is not actionable because of missing required information, which needs to be provided.
Comments
|
Please provide a playground link which shows this. |
dsnet
added
the
WaitingForInfo
mdempsky
changed the title
|
A playground link is ideal, but any way to easily reproduce this would be a good start. |
|
@johnfercher please use/modify this seed program that I derived from your report, to perhaps provide a repro for the problem otherwise this issue isn't actionable package main
import "fmt"
type rect struct {
offsetY float64
footerHeight float64
}
func main() {
self := &rect{
offsetY: 5,
footerHeight: 1,
}
do(self)
}
func do(self *rect) {
height := float64(1)
pageHeight := float64(100)
bottom := float64(10)
top := float64(10)
totalOffsetY := self.offsetY + height + self.footerHeight
maxOffsetPage := pageHeight - bottom - top
greaterThan := totalOffsetY > maxOffsetPage
// At this point totalOffsetY is float64{7}
// maxOffsetPage is float64{80}
// and greaterThan is bool{false}.
fmt.Printf("totalOffsetY: %.2f maxOffsetPage: %.2f\n", totalOffsetY, maxOffsetPage)
if greaterThan == true {
panic("greaterThan is supposed to be false")
}
}Thank you. |
|
Timed out in state WaitingForInfo. Closing. (I am just a bot, though. Please speak up if this is a mistake or you have the requested information.) |
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
What version of Go are you using (
go version)?Does this issue reproduce with the latest release?
No.
What operating system and processor architecture are you using (
go env)?go envOutputWhat did you do?
The greaterThan comparison is returning true.
What did you expect to see?
false must be false
What did you see instead?
false is true
The text was updated successfully, but these errors were encountered: