diff mbox

Set goarch to ppc64le where needed for gccgo testing

Message ID 546D03D2.8020906@linux.vnet.ibm.com
State New
Headers show

Commit Message

Lynn A. Boger Nov. 19, 2014, 8:55 p.m. UTC 
Updated patch:


On 11/19/2014 09:01 AM, Lynn A. Boger wrote:
> Hi,
>
> This change goes along with the change to the GOARCH setting in gccgo 
> for ppc64le which will be done in gofrontend.  The description for 
> that change is here: 
> https://groups.google.com/forum/#!topic/gofrontend-dev/ocEttrpsw-s
>
> This change has been bootstrapped and tested along with the above 
> change to gofrontend on ppc, ppc64, and ppc64le.
>
> 2014-11-19  Lynn Boger <laboger@linux.vnet.ibm.com>
>         * gcc/testsuite/go.test/go-test.exp:  Add case for ppc64le 
> goarch value for go testing
>
>
> Index: gcc/testsuite/go.test/go-test.exp
> ===================================================================
> --- gcc/testsuite/go.test/go-test.exp   (revision 217507)
> +++ gcc/testsuite/go.test/go-test.exp   (working copy)
> @@ -237,13 +237,15 @@ proc go-set-goarch { } {
>                 return ""
>             }
>         }
> -       "powerpc*-*-*" {
> -           if [check_effective_target_ilp32] {
> -               set goarch "ppc"
> -           } else {
> -               set goarch "ppc64"
> -           }
> +       "powerpc-*-*" {
> +           set goarch "ppc"
>         }
> +       "powerpc64-*-*" {
> +           set goarch "ppc64"
> +       }
> +       "powerpc64le-*-*" {
> +           set goarch "ppc64le"
> +       }
>         "s390-*-*" {
>             set goarch "s390"
>         }
>
>
>

Comments

Ian Lance Taylor Nov. 21, 2014, 6:25 p.m. UTC | #1 
On Wed, Nov 19, 2014 at 12:55 PM, Lynn A. Boger
<laboger@linux.vnet.ibm.com> wrote:
> Updated patch:

Thanks.  Committed.

Ian


> On 11/19/2014 09:01 AM, Lynn A. Boger wrote:
>>
>> Hi,
>>
>> This change goes along with the change to the GOARCH setting in gccgo for
>> ppc64le which will be done in gofrontend.  The description for that change
>> is here: https://groups.google.com/forum/#!topic/gofrontend-dev/ocEttrpsw-s
>>
>> This change has been bootstrapped and tested along with the above change
>> to gofrontend on ppc, ppc64, and ppc64le.
>>
>> 2014-11-19  Lynn Boger <laboger@linux.vnet.ibm.com>
>>         * gcc/testsuite/go.test/go-test.exp:  Add case for ppc64le goarch
>> value for go testing
>>
>>
>> Index: gcc/testsuite/go.test/go-test.exp
>> ===================================================================
>> --- gcc/testsuite/go.test/go-test.exp   (revision 217507)
>> +++ gcc/testsuite/go.test/go-test.exp   (working copy)
>> @@ -237,13 +237,15 @@ proc go-set-goarch { } {
>>                 return ""
>>             }
>>         }
>> -       "powerpc*-*-*" {
>> -           if [check_effective_target_ilp32] {
>> -               set goarch "ppc"
>> -           } else {
>> -               set goarch "ppc64"
>> -           }
>> +       "powerpc-*-*" {
>> +           set goarch "ppc"
>>         }
>> +       "powerpc64-*-*" {
>> +           set goarch "ppc64"
>> +       }
>> +       "powerpc64le-*-*" {
>> +           set goarch "ppc64le"
>> +       }
>>         "s390-*-*" {
>>             set goarch "s390"
>>         }
>>
>>
>>
>
diff mbox

Patch

Index: gcc/testsuite/go.test/go-test.exp
===================================================================
--- gcc/testsuite/go.test/go-test.exp   (revision 217768)
+++ gcc/testsuite/go.test/go-test.exp   (working copy)
@@ -241,7 +241,11 @@  proc go-set-goarch { } {
             if [check_effective_target_ilp32] {
                 set goarch "ppc"
             } else {
-               set goarch "ppc64"
+               if [istarget "powerpc64le-*-*"] {
+                   set goarch "ppc64le"
+               } else {
+                   set goarch "ppc64"
+               }
             }
         }
         "s390-*-*" {