Github messages for voidlinux
 help / color / mirror / Atom feed
* [ISSUE] All versions of 32-ARM Go compliers use software floating point.
@ 2021-05-13  4:49 ejolson2005
  2021-05-13  5:14 ` All versions of 32-bit ARM " ejolson2005
                   ` (4 more replies)
  0 siblings, 5 replies; 6+ messages in thread
From: ejolson2005 @ 2021-05-13  4:49 UTC (permalink / raw)
  To: ml

[-- Attachment #1: Type: text/plain, Size: 1030 bytes --]

New issue by ejolson2005 on void-packages repository

https://github.com/void-linux/void-packages/issues/30827

Description:
<!-- Don't request update of package. We have a script for that. https://alpha.de.repo.voidlinux.org/void-updates/void-updates.txt . However, a quality pull request may help. -->
### System
Void 32-bit ARMv6 and ARMv7 versions of the Go compiler.  
### Expected behavior
The Go compiler will use hardware float by default for 32-bit ARMv7 platform and maybe even the ARMv6 platform.
### Actual behavior
The Go compiler packages are built without specifying GOARM=7 or GOARM=6 and apparently default to ARMv5 compatibility.
### Steps to reproduce the behavior
Compile any program that uses floating point and check performance.
### Possible fix
The Alpine Linux build scripts case things as
```
armel)  export GOARCH="arm" GOARM=5 ;;
armhf)  export GOARCH="arm" GOARM=6 ;;
armv7)  export GOARCH="arm" GOARM=7 ;;
```
which seems like a good idea to avoid everything defaulting to GOARM=5.


^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: All versions of 32-bit ARM Go compliers use software floating point.
  2021-05-13  4:49 [ISSUE] All versions of 32-ARM Go compliers use software floating point ejolson2005
@ 2021-05-13  5:14 ` ejolson2005
  2021-05-13  6:08 ` ericonr
                   ` (3 subsequent siblings)
  4 siblings, 0 replies; 6+ messages in thread
From: ejolson2005 @ 2021-05-13  5:14 UTC (permalink / raw)
  To: ml

[-- Attachment #1: Type: text/plain, Size: 292 bytes --]

New comment by ejolson2005 on void-packages repository

https://github.com/void-linux/void-packages/issues/30827#issuecomment-840305562

Comment:
Another work around is to set the environment variable
```
$ export GOARM=7
```
before compiling anything on the ARMv7 version of Void Linux.

^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: All versions of 32-bit ARM Go compliers use software floating point.
  2021-05-13  4:49 [ISSUE] All versions of 32-ARM Go compliers use software floating point ejolson2005
  2021-05-13  5:14 ` All versions of 32-bit ARM " ejolson2005
@ 2021-05-13  6:08 ` ericonr
  2021-05-14 17:29 ` ericonr
                   ` (2 subsequent siblings)
  4 siblings, 0 replies; 6+ messages in thread
From: ericonr @ 2021-05-13  6:08 UTC (permalink / raw)
  To: ml

[-- Attachment #1: Type: text/plain, Size: 273 bytes --]

New comment by ericonr on void-packages repository

https://github.com/void-linux/void-packages/issues/30827#issuecomment-840330685

Comment:
That should definitely be fixed. You can look into `common/environment/build-style/go.sh` to see how the setup works and tweak it.

^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: All versions of 32-bit ARM Go compliers use software floating point.
  2021-05-13  4:49 [ISSUE] All versions of 32-ARM Go compliers use software floating point ejolson2005
  2021-05-13  5:14 ` All versions of 32-bit ARM " ejolson2005
  2021-05-13  6:08 ` ericonr
@ 2021-05-14 17:29 ` ericonr
  2022-05-19  2:16 ` github-actions
  2022-05-19 14:52 ` [ISSUE] [CLOSED] " ericonr
  4 siblings, 0 replies; 6+ messages in thread
From: ericonr @ 2021-05-14 17:29 UTC (permalink / raw)
  To: ml

[-- Attachment #1: Type: text/plain, Size: 324 bytes --]

New comment by ericonr on void-packages repository

https://github.com/void-linux/void-packages/issues/30827#issuecomment-841394147

Comment:
Hm, are you sure?

https://github.com/void-linux/void-packages/blob/6387c58f4e77b52f06280ef2aa5191160a5f67d4/common/environment/build-style/go.sh#L23-L25

How did you test this?

^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: All versions of 32-bit ARM Go compliers use software floating point.
  2021-05-13  4:49 [ISSUE] All versions of 32-ARM Go compliers use software floating point ejolson2005
                   ` (2 preceding siblings ...)
  2021-05-14 17:29 ` ericonr
@ 2022-05-19  2:16 ` github-actions
  2022-05-19 14:52 ` [ISSUE] [CLOSED] " ericonr
  4 siblings, 0 replies; 6+ messages in thread
From: github-actions @ 2022-05-19  2:16 UTC (permalink / raw)
  To: ml

[-- Attachment #1: Type: text/plain, Size: 293 bytes --]

New comment by github-actions[bot] on void-packages repository

https://github.com/void-linux/void-packages/issues/30827#issuecomment-1131014690

Comment:
Issues become stale 90 days after last activity and are closed 14 days after that.  If this issue is still relevant bump it or assign it.

^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: [ISSUE] [CLOSED] All versions of 32-bit ARM Go compliers use software floating point.
  2021-05-13  4:49 [ISSUE] All versions of 32-ARM Go compliers use software floating point ejolson2005
                   ` (3 preceding siblings ...)
  2022-05-19  2:16 ` github-actions
@ 2022-05-19 14:52 ` ericonr
  4 siblings, 0 replies; 6+ messages in thread
From: ericonr @ 2022-05-19 14:52 UTC (permalink / raw)
  To: ml

[-- Attachment #1: Type: text/plain, Size: 1033 bytes --]

Closed issue by ejolson2005 on void-packages repository

https://github.com/void-linux/void-packages/issues/30827

Description:
<!-- Don't request update of package. We have a script for that. https://alpha.de.repo.voidlinux.org/void-updates/void-updates.txt . However, a quality pull request may help. -->
### System
Void 32-bit ARMv6 and ARMv7 versions of the Go compiler.  
### Expected behavior
The Go compiler will use hardware float by default for 32-bit ARMv7 platform and maybe even the ARMv6 platform.
### Actual behavior
The Go compiler packages are built without specifying GOARM=7 or GOARM=6 and apparently default to ARMv5 compatibility.
### Steps to reproduce the behavior
Compile any program that uses floating point and check performance.
### Possible fix
The Alpine Linux build scripts case things as
```
armel)  export GOARCH="arm" GOARM=5 ;;
armhf)  export GOARCH="arm" GOARM=6 ;;
armv7)  export GOARCH="arm" GOARM=7 ;;
```
which seems like a good idea to avoid everything defaulting to GOARM=5.


^ permalink raw reply	[flat|nested] 6+ messages in thread

end of thread, other threads:[~2022-05-19 14:52 UTC | newest]

Thread overview: 6+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2021-05-13  4:49 [ISSUE] All versions of 32-ARM Go compliers use software floating point ejolson2005
2021-05-13  5:14 ` All versions of 32-bit ARM " ejolson2005
2021-05-13  6:08 ` ericonr
2021-05-14 17:29 ` ericonr
2022-05-19  2:16 ` github-actions
2022-05-19 14:52 ` [ISSUE] [CLOSED] " ericonr

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).