Summary: | libgo/math test fails on s390x (undefined symbols cosh, sinh, tanh, hasVX) | ||
---|---|---|---|
Product: | gcc | Reporter: | Dominik Vogt <vogt> |
Component: | go | Assignee: | Ian Lance Taylor <ian> |
Status: | RESOLVED FIXED | ||
Severity: | normal | CC: | krebbel |
Priority: | P3 | ||
Version: | 7.0 | ||
Target Milestone: | --- | ||
Host: | s390x | Target: | s390x |
Build: | Known to work: | ||
Known to fail: | Last reconfirmed: | ||
Attachments: |
Possible patch
crypto patch |
Description
Dominik Vogt
2017-02-09 16:00:28 UTC
Created attachment 40707 [details]
Possible patch
Can you check whether this patch fixes the problem? Thanks.
Yes, that fixes it. Now there's another one in crypto/sha256. Do you want me to open another bug report for that? -- fallback_test.go:19:5: error: reference to undefined name 'useAsm' if useAsm == false { ^ fallback_test.go:22:2: error: reference to undefined name 'useAsm' useAsm = false ^ fallback_test.go:23:17: error: reference to undefined name 'useAsm' defer func() { useAsm = true }() ^ FAIL: crypto/sha256 -- Created attachment 40708 [details]
crypto patch
This patch may fix the crypto/sha256 problem.
Any other problems? `make check-target-libgo` should show all of them.
No need to open separate bugs for each one.
No more than that, and the fix works. (Except for the issue with cplx2.go that has been there from the start https://gcc.gnu.org/bugzilla/show_bug.cgi?id=60181). Author: ian Date: Mon Feb 13 17:37:02 2017 New Revision: 245397 URL: https://gcc.gnu.org/viewcvs?rev=245397&root=gcc&view=rev Log: PR go/79443 libgo: fix some s390x tests Add `+build ignore` to ignore tests that rely on s390x assembly code not yet implemented for gccgo. Fixes GCC PR 79443. Reviewed-on: https://go-review.googlesource.com/36909 Modified: trunk/gcc/go/gofrontend/MERGE trunk/libgo/go/crypto/sha256/fallback_test.go trunk/libgo/go/math/export_s390x_test.go Fix committed to mainline. |