]> Cypherpunks repositories - gostls13.git/commit
[release-branch.go1.11] cmd/compile: use correct store types in softfloat
authorCherry Zhang <cherryyz@google.com>
Sat, 17 Nov 2018 03:53:04 +0000 (22:53 -0500)
committerFilippo Valsorda <filippo@golang.org>
Fri, 14 Dec 2018 19:54:21 +0000 (19:54 +0000)
commit5128a419b3abd4ef10475c0c96eede78363dc267
tree8b231dbbcf3a17e19e681d3b3ca6a57e7f4aa8fe
parent928a4b69195f1f408e4cdb6ad4ee41f5f91f4e86
[release-branch.go1.11] cmd/compile: use correct store types in softfloat

When using softfloat, floating point ops are rewritten to integer
ops. The types of store ops were not rewritten. This may lower
to floating point stores, which are problematic. This CL fixes
this by rewriting the store types as well.

This fixes test/fixedbugs/issue28688.go on Wasm. Softfloat mode
is not used by default on Wasm, and it is not needed as Wasm spec
supports floating points. But it is nice to have the correct
types.

Change-Id: Ib5e19e19fa9491b15c2f60320f8724cace5cefb5
Reviewed-on: https://go-review.googlesource.com/c/149965
Run-TryBot: Cherry Zhang <cherryyz@google.com>
TryBot-Result: Gobot Gobot <gobot@golang.org>
Reviewed-by: Keith Randall <khr@golang.org>
(cherry picked from commit 63a3993a336714f95400e3e614064d4ae72995de)
Reviewed-on: https://go-review.googlesource.com/c/151344
Reviewed-by: Filippo Valsorda <filippo@golang.org>
src/cmd/compile/internal/ssa/softfloat.go