[PATCH] [aarch64] Remove obsolete comment about X30

James Greenhalgh james.greenhalgh@arm.com
Tue Jun 19 15:41:00 GMT 2018


On Mon, Jun 18, 2018 at 08:43:04AM -0500, Siddhesh Poyarekar wrote:
> r217431 changed X30 as caller-saved in CALL_USE_REGISTERS because of
> which this comment about X30 not being marked as call-clobbered is no
> longer accurate.

Is the second paragraph is still relevant to how we define EPILOGUE_USES?

Possibly I'd rewrite the comment to explain the behaviour around calls and
how they interact with x30.

It seems like the right thing to do, but I can't shake the feeling we'd be
losing useful information here.

James

> 	* config/aarch64/aarch64.h: Remove obsolete comment.
> ---
> diff --git a/gcc/config/aarch64/aarch64.h b/gcc/config/aarch64/aarch64.h
> index 976f9afae54..df9fb31aa64 100644
> --- a/gcc/config/aarch64/aarch64.h
> +++ b/gcc/config/aarch64/aarch64.h
> @@ -303,15 +303,6 @@ extern unsigned aarch64_architecture_version;
>     register.  GCC internally uses the poly_int variable aarch64_sve_vg
>     instead.  */
>  
> -/* Note that we don't mark X30 as a call-clobbered register.  The idea is
> -   that it's really the call instructions themselves which clobber X30.
> -   We don't care what the called function does with it afterwards.
> -
> -   This approach makes it easier to implement sibcalls.  Unlike normal
> -   calls, sibcalls don't clobber X30, so the register reaches the
> -   called function intact.  EPILOGUE_USES says that X30 is useful
> -   to the called function.  */
> -
>  #define FIXED_REGISTERS					\
>    {							\
>      0, 0, 0, 0,   0, 0, 0, 0,	/* R0 - R7 */		\
> -- 
> 2.14.4
> 



More information about the Gcc-patches mailing list