public inbox for gcc-patches@gcc.gnu.org
 help / color / mirror / Atom feed
* [patch] introduce aarch64 as a Go architecture
@ 2013-11-29 16:34 Matthias Klose
  2013-11-30  9:43 ` Ian Lance Taylor
  0 siblings, 1 reply; 12+ messages in thread
From: Matthias Klose @ 2013-11-29 16:34 UTC (permalink / raw)
  To: GCC Patches; +Cc: Ian Lance Taylor

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

This patch introduces aarch64 as a Go architecture.

  Matthias

[-- Attachment #2: goarch-aarch64.diff --]
[-- Type: text/plain, Size: 1421 bytes --]

# DP: Introduce aarch64 goarch.

--- a/src/libgo/go/go/build/syslist.go
+++ a/src/libgo/go/go/build/syslist.go
@@ -5,4 +5,4 @@
 package build
 
 const goosList = "darwin dragonfly freebsd linux netbsd openbsd plan9 windows solaris "
-const goarchList = "386 amd64 arm alpha m68k mipso32 mipsn32 mipsn64 mipso64 ppc ppc64 sparc sparc64 "
+const goarchList = "386 amd64 arm aarch64 alpha m68k mipso32 mipsn32 mipsn64 mipso64 ppc ppc64 sparc sparc64 "
--- a/src/libgo/configure.ac
+++ a/src/libgo/configure.ac
@@ -172,6 +172,7 @@
 
 dnl N.B. Keep in sync with gcc/testsuite/go.test/go-test.exp (go-set-goarch).
 is_386=no
+is_aarch64=no
 is_alpha=no
 is_arm=no
 is_m68k=no
@@ -183,6 +184,10 @@
 is_x86_64=no
 GOARCH=unknown
 case ${host} in
+  aarch64*-*-*)
+    is_aarch64=yes
+    GOARCH=aarch64
+    ;;
   alpha*-*-*)
     is_alpha=yes
     GOARCH=alpha
@@ -265,6 +270,7 @@
     ;;
 esac
 AM_CONDITIONAL(LIBGO_IS_386, test $is_386 = yes)
+AM_CONDITIONAL(LIBGO_IS_AARCH64, test $is_aarch64 = yes)
 AM_CONDITIONAL(LIBGO_IS_ALPHA, test $is_alpha = yes)
 AM_CONDITIONAL(LIBGO_IS_ARM, test $is_arm = yes)
 AM_CONDITIONAL(LIBGO_IS_M68K, test $is_m68k = yes)
--- a/src/gcc/testsuite/go.test/go-test.exp
+++ a/src/gcc/testsuite/go.test/go-test.exp
@@ -187,6 +187,9 @@
     global target_triplet
 
     switch -glob $target_triplet {
+	"aarch64*-*-*" {
+	    set goarch "aarch64"
+	}
 	"alpha*-*-*" {
 	    set goarch "alpha"
 	}

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

* Re: [patch] introduce aarch64 as a Go architecture
  2013-11-29 16:34 [patch] introduce aarch64 as a Go architecture Matthias Klose
@ 2013-11-30  9:43 ` Ian Lance Taylor
  2013-11-30 10:41   ` Matthias Klose
                     ` (2 more replies)
  0 siblings, 3 replies; 12+ messages in thread
From: Ian Lance Taylor @ 2013-11-30  9:43 UTC (permalink / raw)
  To: Matthias Klose, Michael Hudson-Doyle
  Cc: GCC Patches, golang-dev, gofrontend-dev

I've gotten a patch from Michael Hudson-Doyle to set GOARCH to arm64
on an Aarch64 system (https://codereview.appspot.com/34830045/).  I've
gotten a patch from Matthias Klose to set GOARCH to aarch64 on such a
system (http://gcc.gnu.org/ml/gcc-patches/2013-11/msg03765.html).

I don't care one way or another myself, but we need to pick one.

Ian


On Fri, Nov 29, 2013 at 5:23 AM, Matthias Klose <doko@ubuntu.com> wrote:
> This patch introduces aarch64 as a Go architecture.
>
>   Matthias

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

* Re: [patch] introduce aarch64 as a Go architecture
  2013-11-30  9:43 ` Ian Lance Taylor
@ 2013-11-30 10:41   ` Matthias Klose
  2013-11-30 11:39     ` Andrew Pinski
  2013-11-30 16:46   ` [golang-dev] " Rob Pike
  2013-12-01 19:55   ` Michael Hudson-Doyle
  2 siblings, 1 reply; 12+ messages in thread
From: Matthias Klose @ 2013-11-30 10:41 UTC (permalink / raw)
  To: Ian Lance Taylor, Michael Hudson-Doyle
  Cc: GCC Patches, golang-dev, gofrontend-dev

Please let's pick aarch64. Everybody names it this way, except of course Debian
:-/  If I understand ARM developers correctly, there will be something like
aarch32 in the future (x32 for ARM), and I think you don't want to call it arm32.

  Matthias

Am 29.11.2013 20:22, schrieb Ian Lance Taylor:
> I've gotten a patch from Michael Hudson-Doyle to set GOARCH to arm64
> on an Aarch64 system (https://codereview.appspot.com/34830045/).  I've
> gotten a patch from Matthias Klose to set GOARCH to aarch64 on such a
> system (http://gcc.gnu.org/ml/gcc-patches/2013-11/msg03765.html).
> 
> I don't care one way or another myself, but we need to pick one.
> 
> Ian
> 
> 
> On Fri, Nov 29, 2013 at 5:23 AM, Matthias Klose <doko@ubuntu.com> wrote:
>> This patch introduces aarch64 as a Go architecture.
>>
>>   Matthias

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

* Re: [patch] introduce aarch64 as a Go architecture
  2013-11-30 10:41   ` Matthias Klose
@ 2013-11-30 11:39     ` Andrew Pinski
  2013-12-02 10:30       ` Richard Earnshaw (home)
  0 siblings, 1 reply; 12+ messages in thread
From: Andrew Pinski @ 2013-11-30 11:39 UTC (permalink / raw)
  To: Matthias Klose
  Cc: Ian Lance Taylor, Michael Hudson-Doyle, GCC Patches, golang-dev,
	gofrontend-dev

On Fri, Nov 29, 2013 at 11:34 AM, Matthias Klose <doko@ubuntu.com> wrote:
> Please let's pick aarch64. Everybody names it this way, except of course Debian

And the linux kernel.

> :-/  If I understand ARM developers correctly, there will be something like
> aarch32 in the future (x32 for ARM), and I think you don't want to call it arm32.
>
>   Matthias
>
> Am 29.11.2013 20:22, schrieb Ian Lance Taylor:
>> I've gotten a patch from Michael Hudson-Doyle to set GOARCH to arm64
>> on an Aarch64 system (https://codereview.appspot.com/34830045/).  I've
>> gotten a patch from Matthias Klose to set GOARCH to aarch64 on such a
>> system (http://gcc.gnu.org/ml/gcc-patches/2013-11/msg03765.html).
>>
>> I don't care one way or another myself, but we need to pick one.
>>
>> Ian
>>
>>
>> On Fri, Nov 29, 2013 at 5:23 AM, Matthias Klose <doko@ubuntu.com> wrote:
>>> This patch introduces aarch64 as a Go architecture.
>>>
>>>   Matthias
>

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

* Re: [golang-dev] Re: [patch] introduce aarch64 as a Go architecture
  2013-11-30  9:43 ` Ian Lance Taylor
  2013-11-30 10:41   ` Matthias Klose
@ 2013-11-30 16:46   ` Rob Pike
  2013-12-01 19:55   ` Michael Hudson-Doyle
  2 siblings, 0 replies; 12+ messages in thread
From: Rob Pike @ 2013-11-30 16:46 UTC (permalink / raw)
  To: Ian Lance Taylor
  Cc: Matthias Klose, Michael Hudson-Doyle, GCC Patches, golang-dev,
	gofrontend-dev

aarch64 is arguably correct but not very evocative (to speak
politely). the arm marketing people must be uninterested in
distinguishing their design from other systems.

arm64 is clearer and in line with amd64, but possibly confusing therefore.

aarch64 is perhaps the better choice but arm64 would be fine with me
too. i too don't really care.

-rob

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

* Re: [patch] introduce aarch64 as a Go architecture
  2013-11-30  9:43 ` Ian Lance Taylor
  2013-11-30 10:41   ` Matthias Klose
  2013-11-30 16:46   ` [golang-dev] " Rob Pike
@ 2013-12-01 19:55   ` Michael Hudson-Doyle
       [not found]     ` <CANp9fE_D4j=GUVh3Y6xmNZCWLZ6z8aw-2fa-nccPjp6m0igdEw@mail.gmail.com>
  2013-12-02  9:02     ` Marcus Shawcroft
  2 siblings, 2 replies; 12+ messages in thread
From: Michael Hudson-Doyle @ 2013-12-01 19:55 UTC (permalink / raw)
  To: Ian Lance Taylor, Matthias Klose; +Cc: GCC Patches, golang-dev, gofrontend-dev

Ian Lance Taylor <iant@google.com> writes:

> I've gotten a patch from Michael Hudson-Doyle to set GOARCH to arm64
> on an Aarch64 system (https://codereview.appspot.com/34830045/).  

Haha, go us.

> I've gotten a patch from Matthias Klose to set GOARCH to aarch64 on
> such a system
> (http://gcc.gnu.org/ml/gcc-patches/2013-11/msg03765.html).
>
> I don't care one way or another myself, but we need to pick one.

I don't care too much myself.  AArch64 is more correct but arm64 is more
obvious.  Also plan9/inferno will use arm64 IIUC.

Cheers,
mwh

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

* Re: [golang-dev] Re: [gofrontend-dev] Re: [patch] introduce aarch64 as a Go architecture
       [not found]       ` <CAH7AAvpS+gwG=7K8xOY+CKmZ=K2UodwD9_+yWsLgH9kSx_PErQ@mail.gmail.com>
@ 2013-12-02  0:06         ` Rob Pike
  2013-12-02 10:21           ` Richard Earnshaw (home)
  0 siblings, 1 reply; 12+ messages in thread
From: Rob Pike @ 2013-12-02  0:06 UTC (permalink / raw)
  To: m4dh4tt3r
  Cc: Dave Cheney, Michael Hudson-Doyle, Ian Lance Taylor,
	Matthias Klose, GCC Patches, golang-dev, gofrontend-dev

arm64 it is

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

* Re: [patch] introduce aarch64 as a Go architecture
  2013-12-01 19:55   ` Michael Hudson-Doyle
       [not found]     ` <CANp9fE_D4j=GUVh3Y6xmNZCWLZ6z8aw-2fa-nccPjp6m0igdEw@mail.gmail.com>
@ 2013-12-02  9:02     ` Marcus Shawcroft
  2013-12-02  9:10       ` Andrew Pinski
  1 sibling, 1 reply; 12+ messages in thread
From: Marcus Shawcroft @ 2013-12-02  9:02 UTC (permalink / raw)
  To: Michael Hudson-Doyle
  Cc: Ian Lance Taylor, Matthias Klose, GCC Patches, golang-dev,
	gofrontend-dev

On 1 December 2013 19:55, Michael Hudson-Doyle
<michael.hudson@linaro.org> wrote:
> Ian Lance Taylor <iant@google.com> writes:
>
>> I've gotten a patch from Michael Hudson-Doyle to set GOARCH to arm64
>> on an Aarch64 system (https://codereview.appspot.com/34830045/).
>
> Haha, go us.
>
>> I've gotten a patch from Matthias Klose to set GOARCH to aarch64 on
>> such a system
>> (http://gcc.gnu.org/ml/gcc-patches/2013-11/msg03765.html).
>>
>> I don't care one way or another myself, but we need to pick one.
>
> I don't care too much myself.  AArch64 is more correct but arm64 is more
> obvious.  Also plan9/inferno will use arm64 IIUC.

All the documentation relevant to this architecture uses the term
"aarch64". How is arm64 obvious?

/Marcus

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

* Re: [patch] introduce aarch64 as a Go architecture
  2013-12-02  9:02     ` Marcus Shawcroft
@ 2013-12-02  9:10       ` Andrew Pinski
  2013-12-02 19:53         ` Mike Stump
  0 siblings, 1 reply; 12+ messages in thread
From: Andrew Pinski @ 2013-12-02  9:10 UTC (permalink / raw)
  To: Marcus Shawcroft
  Cc: Michael Hudson-Doyle, Ian Lance Taylor, Matthias Klose,
	GCC Patches, golang-dev, gofrontend-dev

On Mon, Dec 2, 2013 at 1:02 AM, Marcus Shawcroft
<marcus.shawcroft@gmail.com> wrote:
> On 1 December 2013 19:55, Michael Hudson-Doyle
> <michael.hudson@linaro.org> wrote:
>> Ian Lance Taylor <iant@google.com> writes:
>>
>>> I've gotten a patch from Michael Hudson-Doyle to set GOARCH to arm64
>>> on an Aarch64 system (https://codereview.appspot.com/34830045/).
>>
>> Haha, go us.
>>
>>> I've gotten a patch from Matthias Klose to set GOARCH to aarch64 on
>>> such a system
>>> (http://gcc.gnu.org/ml/gcc-patches/2013-11/msg03765.html).
>>>
>>> I don't care one way or another myself, but we need to pick one.
>>
>> I don't care too much myself.  AArch64 is more correct but arm64 is more
>> obvious.  Also plan9/inferno will use arm64 IIUC.
>
> All the documentation relevant to this architecture uses the term
> "aarch64". How is arm64 obvious?

The same reason Linus used arm64:
https://lkml.org/lkml/2012/7/15/133

Thanks,
Andrew Pinski


>
> /Marcus

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

* Re: [golang-dev] Re: [gofrontend-dev] Re: [patch] introduce aarch64 as a Go architecture
  2013-12-02  0:06         ` [golang-dev] Re: [gofrontend-dev] " Rob Pike
@ 2013-12-02 10:21           ` Richard Earnshaw (home)
  0 siblings, 0 replies; 12+ messages in thread
From: Richard Earnshaw (home) @ 2013-12-02 10:21 UTC (permalink / raw)
  To: Rob Pike
  Cc: m4dh4tt3r, Dave Cheney, Michael Hudson-Doyle, Ian Lance Taylor,
	Matthias Klose, GCC Patches, golang-dev, gofrontend-dev





On 2 Dec 2013, at 00:06, "Rob Pike" <r@golang.org> wrote:

> arm64 it is
> 

This is perverse and completely inconsistent with the rest of the gnu tool chain.  It makes no sense at all to me for go to be inconsistent in this way.

R.

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

* Re: [patch] introduce aarch64 as a Go architecture
  2013-11-30 11:39     ` Andrew Pinski
@ 2013-12-02 10:30       ` Richard Earnshaw (home)
  0 siblings, 0 replies; 12+ messages in thread
From: Richard Earnshaw (home) @ 2013-12-02 10:30 UTC (permalink / raw)
  To: Andrew Pinski
  Cc: Matthias Klose, Ian Lance Taylor, Michael Hudson-Doyle,
	GCC Patches, golang-dev, gofrontend-dev



On 29 Nov 2013, at 19:38, "Andrew Pinski" <pinskia@gmail.com> wrote:

> On Fri, Nov 29, 2013 at 11:34 AM, Matthias Klose <doko@ubuntu.com> wrote:
>> Please let's pick aarch64. Everybody names it this way, except of course Debian
> 
> And the linux kernel.
> 

The Linux kernel reports aarch64 in its uname.  It's only the source tree that uses arm64.

Arm64 is also potentially problematic in regexp strings as it matches arm6*, which was used for some early ARM chips.

R.

>> :-/  If I understand ARM developers correctly, there will be something like
>> aarch32 in the future (x32 for ARM), and I think you don't want to call it arm32.
>> 
>>  Matthias
>> 
>> Am 29.11.2013 20:22, schrieb Ian Lance Taylor:
>>> I've gotten a patch from Michael Hudson-Doyle to set GOARCH to arm64
>>> on an Aarch64 system (https://codereview.appspot.com/34830045/).  I've
>>> gotten a patch from Matthias Klose to set GOARCH to aarch64 on such a
>>> system (http://gcc.gnu.org/ml/gcc-patches/2013-11/msg03765.html).
>>> 
>>> I don't care one way or another myself, but we need to pick one.
>>> 
>>> Ian
>>> 
>>> 
>>> On Fri, Nov 29, 2013 at 5:23 AM, Matthias Klose <doko@ubuntu.com> wrote:
>>>> This patch introduces aarch64 as a Go architecture.
>>>> 
>>>>  Matthias
>> 
> 

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

* Re: [patch] introduce aarch64 as a Go architecture
  2013-12-02  9:10       ` Andrew Pinski
@ 2013-12-02 19:53         ` Mike Stump
  0 siblings, 0 replies; 12+ messages in thread
From: Mike Stump @ 2013-12-02 19:53 UTC (permalink / raw)
  To: Andrew Pinski
  Cc: Marcus Shawcroft, Michael Hudson-Doyle, Ian Lance Taylor,
	Matthias Klose, GCC Patches, golang-dev, gofrontend-dev

On Dec 2, 2013, at 1:10 AM, Andrew Pinski <pinskia@gmail.com> wrote:
>> All the documentation relevant to this architecture uses the term
>> "aarch64". How is arm64 obvious?
> 
> The same reason Linus used arm64:
> https://lkml.org/lkml/2012/7/15/133

Thanks for the link, ah, now I exactly understand what that port is!  :-)  arm64 conveys more to me, more quickly.

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

end of thread, other threads:[~2013-12-02 19:53 UTC | newest]

Thread overview: 12+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2013-11-29 16:34 [patch] introduce aarch64 as a Go architecture Matthias Klose
2013-11-30  9:43 ` Ian Lance Taylor
2013-11-30 10:41   ` Matthias Klose
2013-11-30 11:39     ` Andrew Pinski
2013-12-02 10:30       ` Richard Earnshaw (home)
2013-11-30 16:46   ` [golang-dev] " Rob Pike
2013-12-01 19:55   ` Michael Hudson-Doyle
     [not found]     ` <CANp9fE_D4j=GUVh3Y6xmNZCWLZ6z8aw-2fa-nccPjp6m0igdEw@mail.gmail.com>
     [not found]       ` <CAH7AAvpS+gwG=7K8xOY+CKmZ=K2UodwD9_+yWsLgH9kSx_PErQ@mail.gmail.com>
2013-12-02  0:06         ` [golang-dev] Re: [gofrontend-dev] " Rob Pike
2013-12-02 10:21           ` Richard Earnshaw (home)
2013-12-02  9:02     ` Marcus Shawcroft
2013-12-02  9:10       ` Andrew Pinski
2013-12-02 19:53         ` Mike Stump

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 read-only IMAP folder(s) and NNTP newsgroup(s).