https://gcc.gnu.org/g:2865719efb16e9f199b332fcf06d69c98928738e
commit r15-3377-g2865719efb16e9f199b332fcf06d69c98928738e Author: Richard Sandiford <richard.sandif...@arm.com> Date: Mon Sep 2 09:56:56 2024 +0100 Rename gimple_asm_input_p to gimple_asm_basic_p Following on from the earlier tree rename, this patch renames gimple_asm_input_p to gimple_asm_basic_p, and similarly for related names. gcc/ * doc/gimple.texi (gimple_asm_basic_p): Document. (gimple_asm_set_basic): Likewise. * gimple.h (GF_ASM_INPUT): Rename to... (GF_ASM_BASIC): ...this. (gimple_asm_set_input): Rename to... (gimple_asm_set_basic): ...this. (gimple_asm_input_p): Rename to... (gimple_asm_basic_p): ...this. * cfgexpand.cc (expand_asm_stmt): Update after above renaming. * gimple.cc (gimple_asm_clobbers_memory_p): Likewise. * gimplify.cc (gimplify_asm_expr): Likewise. * ipa-icf-gimple.cc (func_checker::compare_gimple_asm): Likewise. * tree-cfg.cc (stmt_can_terminate_bb_p): Likewise. Diff: --- gcc/cfgexpand.cc | 2 +- gcc/doc/gimple.texi | 9 +++++++++ gcc/gimple.cc | 2 +- gcc/gimple.h | 19 ++++++++++--------- gcc/gimplify.cc | 2 +- gcc/ipa-icf-gimple.cc | 2 +- gcc/tree-cfg.cc | 2 +- 7 files changed, 24 insertions(+), 14 deletions(-) diff --git a/gcc/cfgexpand.cc b/gcc/cfgexpand.cc index 13f8c08d295a..f32cf1b20c9a 100644 --- a/gcc/cfgexpand.cc +++ b/gcc/cfgexpand.cc @@ -3121,7 +3121,7 @@ expand_asm_stmt (gasm *stmt) location_t locus = gimple_location (stmt); - if (gimple_asm_input_p (stmt)) + if (gimple_asm_basic_p (stmt)) { const char *s = gimple_asm_string (stmt); tree string = build_string (strlen (s), s); diff --git a/gcc/doc/gimple.texi b/gcc/doc/gimple.texi index 5f241b1c64f4..d8aaca260493 100644 --- a/gcc/doc/gimple.texi +++ b/gcc/doc/gimple.texi @@ -1112,6 +1112,15 @@ Return the string representing the assembly instruction in @code{GIMPLE_ASM} @code{G}. @end deftypefn +@deftypefn {GIMPLE function} bool gimple_asm_basic_p (const gasm *g) +Return true if @code{G} is a basic asm rather than an extended asm. +@end deftypefn + +@deftypefn {GIMPLE function} void gimple_asm_set_basic (gasm *g, bool basic_p) +Mark asm statement @code{G} as a basic asm or an extended asm based on +@code{BASIC_P}. +@end deftypefn + @deftypefn {GIMPLE function} bool gimple_asm_volatile_p (const gasm *g) Return true if @code{G} is an asm statement marked volatile. @end deftypefn diff --git a/gcc/gimple.cc b/gcc/gimple.cc index a9f968cb0389..6e28cf291e16 100644 --- a/gcc/gimple.cc +++ b/gcc/gimple.cc @@ -2944,7 +2944,7 @@ gimple_asm_clobbers_memory_p (const gasm *stmt) } /* Non-empty basic ASM implicitly clobbers memory. */ - if (gimple_asm_input_p (stmt) && strlen (gimple_asm_string (stmt)) != 0) + if (gimple_asm_basic_p (stmt) && strlen (gimple_asm_string (stmt)) != 0) return true; return false; diff --git a/gcc/gimple.h b/gcc/gimple.h index bd315ffc2dd4..ee986eaf1539 100644 --- a/gcc/gimple.h +++ b/gcc/gimple.h @@ -135,7 +135,7 @@ enum gimple_rhs_class Keep this list sorted. */ enum gf_mask { - GF_ASM_INPUT = 1 << 0, + GF_ASM_BASIC = 1 << 0, GF_ASM_VOLATILE = 1 << 1, GF_ASM_INLINE = 1 << 2, GF_CALL_FROM_THUNK = 1 << 0, @@ -4227,24 +4227,25 @@ gimple_asm_set_inline (gasm *asm_stmt, bool inline_p) } -/* If INPUT_P is true, mark asm ASM_STMT as an ASM_INPUT. */ +/* Mark whether asm ASM_STMT is a basic asm or an extended asm, based on + BASIC_P. */ inline void -gimple_asm_set_input (gasm *asm_stmt, bool input_p) +gimple_asm_set_basic (gasm *asm_stmt, bool basic_p) { - if (input_p) - asm_stmt->subcode |= GF_ASM_INPUT; + if (basic_p) + asm_stmt->subcode |= GF_ASM_BASIC; else - asm_stmt->subcode &= ~GF_ASM_INPUT; + asm_stmt->subcode &= ~GF_ASM_BASIC; } -/* Return true if asm ASM_STMT is an ASM_INPUT. */ +/* Return true if asm ASM_STMT is a basic asm rather than an extended asm. */ inline bool -gimple_asm_input_p (const gasm *asm_stmt) +gimple_asm_basic_p (const gasm *asm_stmt) { - return (asm_stmt->subcode & GF_ASM_INPUT) != 0; + return (asm_stmt->subcode & GF_ASM_BASIC) != 0; } diff --git a/gcc/gimplify.cc b/gcc/gimplify.cc index f0edb4f7edd9..9300138aa0c7 100644 --- a/gcc/gimplify.cc +++ b/gcc/gimplify.cc @@ -7352,7 +7352,7 @@ gimplify_asm_expr (tree *expr_p, gimple_seq *pre_p, gimple_seq *post_p) ASM_VOLATILE_P (expr) || noutputs == 0 || labels); - gimple_asm_set_input (stmt, ASM_BASIC_P (expr)); + gimple_asm_set_basic (stmt, ASM_BASIC_P (expr)); gimple_asm_set_inline (stmt, ASM_INLINE_P (expr)); gimplify_seq_add_stmt (pre_p, stmt); diff --git a/gcc/ipa-icf-gimple.cc b/gcc/ipa-icf-gimple.cc index 4c3174b68b67..31f6cdbda060 100644 --- a/gcc/ipa-icf-gimple.cc +++ b/gcc/ipa-icf-gimple.cc @@ -986,7 +986,7 @@ func_checker::compare_gimple_asm (const gasm *g1, const gasm *g2) if (gimple_asm_volatile_p (g1) != gimple_asm_volatile_p (g2)) return false; - if (gimple_asm_input_p (g1) != gimple_asm_input_p (g2)) + if (gimple_asm_basic_p (g1) != gimple_asm_basic_p (g2)) return false; if (gimple_asm_inline_p (g1) != gimple_asm_inline_p (g2)) diff --git a/gcc/tree-cfg.cc b/gcc/tree-cfg.cc index e6fd1294b958..fcb488d87113 100644 --- a/gcc/tree-cfg.cc +++ b/gcc/tree-cfg.cc @@ -8887,7 +8887,7 @@ stmt_can_terminate_bb_p (gimple *t) } if (gasm *asm_stmt = dyn_cast <gasm *> (t)) - if (gimple_asm_volatile_p (asm_stmt) || gimple_asm_input_p (asm_stmt)) + if (gimple_asm_volatile_p (asm_stmt) || gimple_asm_basic_p (asm_stmt)) return true; return false;