https://gcc.gnu.org/bugzilla/show_bug.cgi?id=63269
Eric Gallager changed:
What|Removed |Added
Status|WAITING |RESOLVED
URL|
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=63269
--- Comment #6 from Dominik Vogt ---
(In reply to Ian Lance Taylor from comment #3)
> First, let me say that this code is in the Go master library and must be
> fixed there. It might be more effective to discuss it on the Go issue
> tracker at h
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=63269
--- Comment #5 from Dominik Vogt ---
regarding 1)
My earlier explanation of the problem was wrong. Multiply and add is not
generated; it probably only was in the artificial test case that I made and
certainly did not compile with -ffp-contract=
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=63269
--- Comment #4 from Dominik Vogt ---
regarding 2)
> I'm not entirely persuaded by your argument for item 2. ...
Hm, good that you doubted it, because the actual mistake is somehwere else:
The unpatched code has
if l != float64(i)
but if yo
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=63269
Ian Lance Taylor changed:
What|Removed |Added
Status|UNCONFIRMED |WAITING
Last reconfirmed|
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=63269
--- Comment #2 from Dominik Vogt ---
Created attachment 33506
--> https://gcc.gnu.org/bugzilla/attachment.cgi?id=33506&action=edit
updated patch
Managed to get a stray character into the test between testing and committing
it. Fixed.
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=63269
--- Comment #1 from Dominik Vogt ---
Created attachment 33493
--> https://gcc.gnu.org/bugzilla/attachment.cgi?id=33493&action=edit
Proposed fixes