Re: [PATCH] gcc/powerpc: remove -many

Re: [PATCH] gcc/powerpc: remove -many

[Alan Modra]

On Wed, May 05, 2010 at 08:58:51AM -0700, Nathan Froyd wrote:
> We've had the patch below in our tree for a while; I think it does what
> you want.
> 
> Alan, OK for binutils?

No.  This patch appears to be against an old version of binutils.
parse_cpu is now in opcodes/ppc-dis.c

> 	gas/
> 	* config/tc-ppc.c (parse_cpu): Handle -mpmr option.
> 
> Index: gas/config/tc-ppc.c
> ===================================================================
> --- gas/config/tc-ppc.c	(revision 181279)
> +++ gas/config/tc-ppc.c	(revision 181280)
> @@ -864,6 +864,10 @@ parse_cpu (const char *arg)
>      }
>    /* -mppc64 and -m620 mean to assemble for the 64-bit PowerPC
>       620.  */
> +  else if (strcmp (arg, "pmr") == 0)
> +    {
> +      ppc_cpu |= PPC_OPCODE_PMR;
> +    }
>    else if (strcmp (arg, "ppc64") == 0 || strcmp (arg, "620") == 0)
>      {
>        ppc_cpu = PPC_OPCODE_PPC | PPC_OPCODE_CLASSIC | PPC_OPCODE_64;

-- 
Alan Modra
Australia Development Lab, IBM

Re: [PATCH] gcc/powerpc: remove -many

[Sebastian Andrzej Siewior]

* Alan Modra | 2010-05-06 09:07:59 [+0930]:

>On Wed, May 05, 2010 at 08:58:51AM -0700, Nathan Froyd wrote:
>> We've had the patch below in our tree for a while; I think it does what
>> you want.
>> 
>> Alan, OK for binutils?
>
>No.  This patch appears to be against an old version of binutils.
>parse_cpu is now in opcodes/ppc-dis.c

Alan, I haven't found any follow on this.
So you prefer adding -mpmr to binutils and gcc to relevant cpus or do
you prefer to have -mpmr and a patch to the linux kernel to add this
option once PMR is activated?

Sebastian