Patchwork SPARC: Clarify -mapp-regs option

login
register
mail settings
Submitter Sebastian Huber
Date March 8, 2014, 9:51 a.m.
Message ID <1394272306-9748-1-git-send-email-sebastian.huber@embedded-brains.de>
Download mbox | patch
Permalink /patch/328171/
State New
Headers show

Comments

Sebastian Huber - March 8, 2014, 9:51 a.m.
gcc/ChangeLog
2014-03-08  Sebastian Huber  <sebastian.huber@embedded-brains.de>

	* doc/invoke.texi (mapp-regs): Clarify.
---
 gcc/doc/invoke.texi | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)
Eric Botcazou - March 15, 2014, 9:51 p.m.
> gcc/ChangeLog
> 2014-03-08  Sebastian Huber  <sebastian.huber@embedded-brains.de>
> 
> 	* doc/invoke.texi (mapp-regs): Clarify.

OK modulo:

> diff --git a/gcc/doc/invoke.texi b/gcc/doc/invoke.texi
> index 2ee091a..12b43fa 100644
> --- a/gcc/doc/invoke.texi
> +++ b/gcc/doc/invoke.texi
> @@ -20818,7 +20818,8 @@ These @samp{-m} options are supported on the SPARC:
>  @opindex mapp-regs
>  Specify @option{-mapp-regs} to generate output using the global registers
>  2 through 4, which the SPARC SVR4 ABI reserves for applications.  This
> -is the default.
> +is the default.  Like the global register 1 the global registers 2 through4
> +are used as volatile registers (may be destroyed across a function
> call).

I personally don't like the "volatile" wording much, please reuse the wording 
of the -fcall-used-REG option.

Patch

diff --git a/gcc/doc/invoke.texi b/gcc/doc/invoke.texi
index 2ee091a..12b43fa 100644
--- a/gcc/doc/invoke.texi
+++ b/gcc/doc/invoke.texi
@@ -20818,7 +20818,8 @@  These @samp{-m} options are supported on the SPARC:
 @opindex mapp-regs
 Specify @option{-mapp-regs} to generate output using the global registers
 2 through 4, which the SPARC SVR4 ABI reserves for applications.  This
-is the default.
+is the default.  Like the global register 1 the global registers 2 through 4
+are used as volatile registers (may be destroyed across a function call).
 
 To be fully SVR4 ABI-compliant at the cost of some performance loss,
 specify @option{-mno-app-regs}.  You should compile libraries and system