Hi Richard! Thanks for the review.
I just noted there already were index entries for:
crtl->args.pops_args
crtl->args.pretend_args_size
crtl->outgoing_args_size
And sticked to the pattern.
Upon closer inspection today, it seems that two entries were
declared for each of the variables above, for example:
@findex outgoing_args_size
@findex crtl->outgoing_args_size
@findex pretend_args_size
@findex crtl->args.pretend_args_size
@findex pops_args
@findex crtl->args.pops_args
Would you like me to add the second `@findex` entry as well?
Best regards,
Léo
Em 2026-06-08 06:38, Richard Biener escreveu:
On Sat, Jun 6, 2026 at 10:02 PM Léo Hardt <[email protected]> wrote:
Good evening!
The variables 'current_function_is_leaf' and
'current_function_uses_only_leaf_args' were removed
on commit 416ff32e, and now are part of the rtl_data struct.
This commit is to clean up the documentation that
mentions these variables.
LGTM, but ...
gcc/ChangeLog:
* doc/tm.texi: Update references to global variables that no longer
exist.
* doc/tm.texi.in: Likewise.
Signed-off-by: Léo Hardt <[email protected]>
---
gcc/doc/tm.texi | 12 ++++++------
gcc/doc/tm.texi.in | 12 ++++++------
2 files changed, 12 insertions(+), 12 deletions(-)
diff --git a/gcc/doc/tm.texi b/gcc/doc/tm.texi
index d1e5d1b79..f1d7eb9af 100644
--- a/gcc/doc/tm.texi
+++ b/gcc/doc/tm.texi
@@ -2378,17 +2378,17 @@ treatment of leaf functions, and registers need to
be
renumbered to do
this.
@end defmac
-@findex current_function_is_leaf
-@findex current_function_uses_only_leaf_regs
+@findex crtl->is_leaf
+@findex crtl->uses_only_leaf_regs
... do we need to add markup because of the ->? Should the index refer
to 'is_leaf'/'uses_only_leaf_regs' without specifying the crtl namespace?
@code{TARGET_ASM_FUNCTION_PROLOGUE} and
@code{TARGET_ASM_FUNCTION_EPILOGUE} must usually treat leaf functions
-specially. They can test the C variable @code{current_function_is_leaf}
-which is nonzero for leaf functions. @code{current_function_is_leaf} is
+specially. They can test the C variable @code{crtl->is_leaf}
+which is nonzero for leaf functions. @code{crtl->is_leaf} is
set prior to local register allocation and is valid for the remaining
compiler passes. They can also test the C variable
-@code{current_function_uses_only_leaf_regs} which is nonzero for leaf
+@code{crtl->uses_only_leaf_regs} which is nonzero for leaf
functions which only use leaf registers.
-@code{current_function_uses_only_leaf_regs} is valid after all passes
+@code{crtl->uses_only_leaf_regs} is valid after all passes
that modify the instructions have been run and is only useful if
@code{LEAF_REGISTERS} is defined.
@c changed this to fix overfull. ALSO: why the "it" at the beginning
diff --git a/gcc/doc/tm.texi.in b/gcc/doc/tm.texi.in
index 6c9c4658b..8a275545b 100644
--- a/gcc/doc/tm.texi.in
+++ b/gcc/doc/tm.texi.in
@@ -1963,17 +1963,17 @@ treatment of leaf functions, and registers need to
be
renumbered to do
this.
@end defmac
-@findex current_function_is_leaf
-@findex current_function_uses_only_leaf_regs
+@findex crtl->is_leaf
+@findex crtl->uses_only_leaf_regs
@code{TARGET_ASM_FUNCTION_PROLOGUE} and
@code{TARGET_ASM_FUNCTION_EPILOGUE} must usually treat leaf functions
-specially. They can test the C variable @code{current_function_is_leaf}
-which is nonzero for leaf functions. @code{current_function_is_leaf} is
+specially. They can test the C variable @code{crtl->is_leaf}
+which is nonzero for leaf functions. @code{crtl->is_leaf} is
set prior to local register allocation and is valid for the remaining
compiler passes. They can also test the C variable
-@code{current_function_uses_only_leaf_regs} which is nonzero for leaf
+@code{crtl->uses_only_leaf_regs} which is nonzero for leaf
functions which only use leaf registers.
-@code{current_function_uses_only_leaf_regs} is valid after all passes
+@code{crtl->uses_only_leaf_regs} is valid after all passes
that modify the instructions have been run and is only useful if
@code{LEAF_REGISTERS} is defined.
@c changed this to fix overfull. ALSO: why the "it" at the beginning
--
2.39.5
