FreeChainXenon/gcc
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

Update comment about is_leaf

Browse source 
crtl->is_leaf has a comment above it referring to "Local Register
Allocation". However this is set by IRA and not LRA since the meaning
of Local Register Allocator in GCC has changed quite drastically since
1999 when this comment was introduced above the variable
current_function_is_leaf. From memory gcc in those days had "local"
and "global" register allocation - however all of this has changed
quite a bit drastically since.

This patch merely updates the comment as I did some head scratching
this morning about how this could be set before Local Register
Allocator especially given the comment in ira.c as below.


 /* Determine if the current function is a leaf before running IRA
     since this can impact optimizations done by the prologue and
     epilogue thus changing register elimination offsets.  */
  crtl->is_leaf = leaf_function_p ();

From-SVN: r249223
This commit is contained in:
Ramana Radhakrishnan 2017-06-15 14:17:04 +00:00 • committed by Ramana Radhakrishnan
parent 61f5d85294
commit 55dabf65f0
2 changed files with 6 additions and 1 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

5
gcc/ChangeLog
View file

@ -1,3 +1,8 @@
2017-06-15 Ramana Radhakrishnan <ramana.radhakrishnan@arm.com>
* emit-rtl.h (is_leaf): Update comment about local
register allocator.
2017-06-15 Jozef Lawrynowicz <jozef.l@somniumtech.com>
PR target/78818

2
gcc/emit-rtl.h
View file

@ -267,7 +267,7 @@ struct GTY(()) rtl_data {
/* Nonzero if function being compiled doesn't contain any calls
(ignoring the prologue and epilogue). This is set prior to
local register allocation and is valid for the remaining
register allocation in IRA and is valid for the remaining
compiler passes. */
bool is_leaf;