This is the mail archive of the
binutils@sourceware.org
mailing list for the binutils project.
Re: [PATCH] Accept Altivec instructions when using -mpower6 and -mcell
- From: Peter Bergner <bergner at vnet dot ibm dot com>
- To: binutils at sourceware dot org
- Cc: Andrew Pinski <pinskia at gmail dot com>
- Date: Mon, 28 Jul 2008 21:31:29 -0500
- Subject: Re: [PATCH] Accept Altivec instructions when using -mpower6 and -mcell
- References: <20080728230431.GA14296@vervain.rchland.ibm.com>
On Mon, 2008-07-28 at 18:04 -0500, Peter Bergner wrote:
> Lastly, after attempting to move the POWER4 gas testcase to powerpc*-*-*
> (which I am not doing in this patch), I noticed that the assembler option
> "-a64" was flagged as an illegal option when I configured with:
I'm sorry, but the power4 hunks below should not have made it through.
Please ignore that part of the patch. I didn't fell comfortable with
that change, since if someone configured with --target=powerpc-linux
and did not enable powerpc64-linux, the test would fail...or is the
testsuite designed to be run when we've configured using --enable-targets=all?
Peter
> Index: gas/testsuite/gas/ppc/power4.d
> ===================================================================
> RCS file: /cvs/src/src/gas/testsuite/gas/ppc/power4.d,v
> retrieving revision 1.6
> diff -u -p -r1.6 power4.d
> --- gas/testsuite/gas/ppc/power4.d 2 Mar 2005 13:25:01 -0000 1.6
> +++ gas/testsuite/gas/ppc/power4.d 28 Jul 2008 22:03:04 -0000
> @@ -1,5 +1,5 @@
> #objdump: -drx -Mpower4
> -#as: -mpower4
> +#as: -a64 -mpower4
> #name: Power4 instructions
>
> .*: +file format elf64-powerpc
> Index: gas/testsuite/gas/ppc/ppc.exp
> ===================================================================
> RCS file: /cvs/src/src/gas/testsuite/gas/ppc/ppc.exp,v
> retrieving revision 1.18
> diff -u -p -r1.18 ppc.exp
> --- gas/testsuite/gas/ppc/ppc.exp 30 Apr 2008 06:47:24 -0000 1.18
> +++ gas/testsuite/gas/ppc/ppc.exp 28 Jul 2008 22:03:04 -0000
> @@ -10,8 +10,6 @@ if { [istarget powerpc64*-*-*] || [istar
> run_dump_test "astest64"
> run_dump_test "astest2_64"
> run_dump_test "test1elf64"
> - run_dump_test "power4"
> - run_dump_test "cell"
> run_list_test "range64" "-a64"
> } elseif { [istarget powerpc*-*aix*] } then {
> run_dump_test "test1xcoff32"
> @@ -45,5 +43,8 @@ if { [istarget powerpc*-*-*] } then {
> run_list_test "range" "-a32"
> run_dump_test "ppc750ps"
> run_dump_test "e500mc"
> + run_dump_test "power4"
> + run_dump_test "cell"
> + run_dump_test "power6"
> }
> }