Update comment about is_leaf
authorRamana Radhakrishnan <ramana.radhakrishnan@arm.com>
Thu, 15 Jun 2017 14:17:04 +0000 (14:17 +0000)
committerRamana Radhakrishnan <ramana@gcc.gnu.org>
Thu, 15 Jun 2017 14:17:04 +0000 (14:17 +0000)
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

gcc/ChangeLog
gcc/emit-rtl.h

index 3f2f34417c656660f8167f1dcd6f8f5dba017b3b..b8c2f055b474e9f5b42c6c6301f8fef326cc6aa3 100644 (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
index da60a2d808c1742f65f0571c3422422292163906..b455b4c00fb96f21b6d8e7b09bb1d4d7de526e1e 100644 (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;