public inbox for gcc-patches@gcc.gnu.org
 help / color / mirror / Atom feed
* [PATCH, GCC/ARM] Complete legend for ARM register allocation in arm.h
@ 2017-05-26 14:35 Thomas Preudhomme
  2017-06-05  9:40 ` [PATCH, GCC/ARM, ping] " Thomas Preudhomme
  2017-06-05  9:41 ` [PATCH, GCC/ARM] " Kyrill Tkachov
  0 siblings, 2 replies; 3+ messages in thread
From: Thomas Preudhomme @ 2017-05-26 14:35 UTC (permalink / raw)
  To: Kyrill Tkachov, Ramana Radhakrishnan, Richard Earnshaw, gcc-patches

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

[resending with gcc-patches this time]

Hi,

The comment describing ARM register allocation in arm.h contains 3 keys:
S, F and *. The legend only explains S and *, this patch remedies that.

ChangeLog entry is as follows:

*** gcc/ChangeLog ***

2017-05-24  Thomas Preud'homme  <thomas.preudhomme@arm.com>

	* config/arm/arm.h: explain F symbol found in description of ARM
	register allocation in its legend.

Is this ok for trunk?

Best regards,

Thomas


[-- Attachment #2: complete_arm_register_allocation_key.patch --]
[-- Type: text/x-patch, Size: 481 bytes --]

diff --git a/gcc/config/arm/arm.h b/gcc/config/arm/arm.h
index 1821f48ed7ed45ffa5503541426921ec0d0a3736..d0b448d273cb3683f06512fea80ebf65df587ab1 100644
--- a/gcc/config/arm/arm.h
+++ b/gcc/config/arm/arm.h
@@ -690,7 +690,7 @@ extern int arm_arch_cmse;
 /* Standard register usage.  */
 
 /* Register allocation in ARM Procedure Call Standard
-   (S - saved over call).
+   (S - saved over call, F - Frame-related).
 
 	r0	   *	argument word/integer result
 	r1-r3		argument word


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

* Re: [PATCH, GCC/ARM, ping] Complete legend for ARM register allocation in arm.h
  2017-05-26 14:35 [PATCH, GCC/ARM] Complete legend for ARM register allocation in arm.h Thomas Preudhomme
@ 2017-06-05  9:40 ` Thomas Preudhomme
  2017-06-05  9:41 ` [PATCH, GCC/ARM] " Kyrill Tkachov
  1 sibling, 0 replies; 3+ messages in thread
From: Thomas Preudhomme @ 2017-06-05  9:40 UTC (permalink / raw)
  To: Kyrill Tkachov, Ramana Radhakrishnan, Richard Earnshaw, gcc-patches

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

Ping?

Best regards,

Thomas

On 26/05/17 15:20, Thomas Preudhomme wrote:
> [resending with gcc-patches this time]
>
> Hi,
>
> The comment describing ARM register allocation in arm.h contains 3 keys:
> S, F and *. The legend only explains S and *, this patch remedies that.
>
> ChangeLog entry is as follows:
>
> *** gcc/ChangeLog ***
>
> 2017-05-24  Thomas Preud'homme  <thomas.preudhomme@arm.com>
>
>     * config/arm/arm.h: explain F symbol found in description of ARM
>     register allocation in its legend.
>
> Is this ok for trunk?
>
> Best regards,
>
> Thomas
>

[-- Attachment #2: complete_arm_register_allocation_key.patch --]
[-- Type: text/x-patch, Size: 480 bytes --]

diff --git a/gcc/config/arm/arm.h b/gcc/config/arm/arm.h
index 1821f48ed7ed45ffa5503541426921ec0d0a3736..d0b448d273cb3683f06512fea80ebf65df587ab1 100644
--- a/gcc/config/arm/arm.h
+++ b/gcc/config/arm/arm.h
@@ -690,7 +690,7 @@ extern int arm_arch_cmse;
 /* Standard register usage.  */
 
 /* Register allocation in ARM Procedure Call Standard
-   (S - saved over call).
+   (S - saved over call, F - Frame-related).
 
 	r0	   *	argument word/integer result
 	r1-r3		argument word

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

* Re: [PATCH, GCC/ARM] Complete legend for ARM register allocation in arm.h
  2017-05-26 14:35 [PATCH, GCC/ARM] Complete legend for ARM register allocation in arm.h Thomas Preudhomme
  2017-06-05  9:40 ` [PATCH, GCC/ARM, ping] " Thomas Preudhomme
@ 2017-06-05  9:41 ` Kyrill Tkachov
  1 sibling, 0 replies; 3+ messages in thread
From: Kyrill Tkachov @ 2017-06-05  9:41 UTC (permalink / raw)
  To: Thomas Preudhomme, Ramana Radhakrishnan, Richard Earnshaw, gcc-patches


On 26/05/17 15:20, Thomas Preudhomme wrote:
> [resending with gcc-patches this time]
>
> Hi,
>
> The comment describing ARM register allocation in arm.h contains 3 keys:
> S, F and *. The legend only explains S and *, this patch remedies that.
>
> ChangeLog entry is as follows:
>
> *** gcc/ChangeLog ***
>
> 2017-05-24  Thomas Preud'homme  <thomas.preudhomme@arm.com>
>
>     * config/arm/arm.h: explain F symbol found in description of ARM
>     register allocation in its legend.
>
> Is this ok for trunk?
>

Ok.
Thanks,
Kyrill

> Best regards,
>
> Thomas
>

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

end of thread, other threads:[~2017-06-05  9:41 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2017-05-26 14:35 [PATCH, GCC/ARM] Complete legend for ARM register allocation in arm.h Thomas Preudhomme
2017-06-05  9:40 ` [PATCH, GCC/ARM, ping] " Thomas Preudhomme
2017-06-05  9:41 ` [PATCH, GCC/ARM] " Kyrill Tkachov

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).