diff mbox

[doc,ARM] Deprecate -mapcs and -mapcs-frame

Message ID 54B64721.3050502@arm.com
State New
Headers show

Commit Message

Kyrylo Tkachov Jan. 14, 2015, 10:38 a.m. UTC
Hi all,

-mapcs-frame (and its' alias -mapcs) are somewhat bitrotten and the ABI 
they represent is deprecated anyway so this is a patch to deprecate the 
option. It's not being removed here, just documented as deprecated.

Kyrill

2015-01-14  Kyrylo Tkachov  <kyrylo.tkachov@arm.com>

     * doc/invoke.texi (mapcs): Mention deprecation.
     (mapcs-frame): Likewise.

Comments

Richard Earnshaw Jan. 14, 2015, 11:13 a.m. UTC | #1
On 14/01/15 10:38, Kyrill Tkachov wrote:
> Hi all,
> 
> -mapcs-frame (and its' alias -mapcs) are somewhat bitrotten and the ABI 
> they represent is deprecated anyway so this is a patch to deprecate the 
> option. It's not being removed here, just documented as deprecated.
> 
> Kyrill
> 
> 2015-01-14  Kyrylo Tkachov  <kyrylo.tkachov@arm.com>
> 
>      * doc/invoke.texi (mapcs): Mention deprecation.
>      (mapcs-frame): Likewise.
> 
> 
> arm-mapcs-deprecated.patch
> 
> 
> diff --git a/gcc/doc/invoke.texi b/gcc/doc/invoke.texi
> index d2f3c79..7a72120 100644
> --- a/gcc/doc/invoke.texi
> +++ b/gcc/doc/invoke.texi
> @@ -12683,10 +12683,11 @@ Standard for all functions, even if this is not strictly necessary for
>  correct execution of the code.  Specifying @option{-fomit-frame-pointer}
>  with this option causes the stack frames not to be generated for
>  leaf functions.  The default is @option{-mno-apcs-frame}.
> +This option is deprecated.
>  
>  @item -mapcs
>  @opindex mapcs
> -This is a synonym for @option{-mapcs-frame}.
> +This is a synonym for @option{-mapcs-frame} and is deprecated.
>  
>  @ignore
>  @c not currently implemented
> 

OK.

I think this needs a mention in the release notes as well.

R.
Kyrylo Tkachov Jan. 14, 2015, 11:15 a.m. UTC | #2
On 14/01/15 11:13, Richard Earnshaw wrote:
> On 14/01/15 10:38, Kyrill Tkachov wrote:
>> Hi all,
>>
>> -mapcs-frame (and its' alias -mapcs) are somewhat bitrotten and the ABI
>> they represent is deprecated anyway so this is a patch to deprecate the
>> option. It's not being removed here, just documented as deprecated.
>>
>> Kyrill
>>
>> 2015-01-14  Kyrylo Tkachov  <kyrylo.tkachov@arm.com>
>>
>>       * doc/invoke.texi (mapcs): Mention deprecation.
>>       (mapcs-frame): Likewise.
>>
>>
>> arm-mapcs-deprecated.patch
>>
>>
>> diff --git a/gcc/doc/invoke.texi b/gcc/doc/invoke.texi
>> index d2f3c79..7a72120 100644
>> --- a/gcc/doc/invoke.texi
>> +++ b/gcc/doc/invoke.texi
>> @@ -12683,10 +12683,11 @@ Standard for all functions, even if this is not strictly necessary for
>>   correct execution of the code.  Specifying @option{-fomit-frame-pointer}
>>   with this option causes the stack frames not to be generated for
>>   leaf functions.  The default is @option{-mno-apcs-frame}.
>> +This option is deprecated.
>>   
>>   @item -mapcs
>>   @opindex mapcs
>> -This is a synonym for @option{-mapcs-frame}.
>> +This is a synonym for @option{-mapcs-frame} and is deprecated.
>>   
>>   @ignore
>>   @c not currently implemented
>>
> OK.
>
> I think this needs a mention in the release notes as well.
Thanks,
the wwdocs patch is https://gcc.gnu.org/ml/gcc-patches/2015-01/msg01003.html

Kyrill

>
> R.
diff mbox

Patch

diff --git a/gcc/doc/invoke.texi b/gcc/doc/invoke.texi
index d2f3c79..7a72120 100644
--- a/gcc/doc/invoke.texi
+++ b/gcc/doc/invoke.texi
@@ -12683,10 +12683,11 @@  Standard for all functions, even if this is not strictly necessary for
 correct execution of the code.  Specifying @option{-fomit-frame-pointer}
 with this option causes the stack frames not to be generated for
 leaf functions.  The default is @option{-mno-apcs-frame}.
+This option is deprecated.
 
 @item -mapcs
 @opindex mapcs
-This is a synonym for @option{-mapcs-frame}.
+This is a synonym for @option{-mapcs-frame} and is deprecated.
 
 @ignore
 @c not currently implemented