From e7960be786ae786c6fc2ec48480bfb279c6d99e2 Mon Sep 17 00:00:00 2001 From: Richard Sandiford Date: Sat, 24 Dec 2016 14:30:18 +0000 Subject: [PATCH] Make it cheaper to test whether an SSA name is a virtual operand virtual_operand_p handled SSA names by looking at the flags of the underlying variable. This seems to be a relatively common source of cache misses, mainly because virtual_operand_p is the first thing tested by is_gimple_reg. This patch caches the information in the SSA name itself. Several flags seem to be free so the patch arbitrarily uses public_flag. Tested on aarch64-linux-gnu and x86_64-linux-gnu. It improves compile time by a small (<1%) but reproducable amount on the tests I've tried. gcc/ * tree-core.h (tree_base): Document the meaning of public_flag for SSA names. * tree.h (SSA_NAME_IS_VIRTUAL_OPERAND): New macro. (SET_SSA_NAME_VAR_OR_IDENTIFIER): Record whether the variable is a virtual operand. * gimple-expr.h (virtual_operand_p): Use SSA_NAME_IS_VIRTUAL_OPERAND. From-SVN: r243916 --- gcc/ChangeLog | 9 +++++++++ gcc/gimple-expr.h | 6 +----- gcc/tree-core.h | 3 +++ gcc/tree.h | 16 +++++++++++++++- 4 files changed, 28 insertions(+), 6 deletions(-) diff --git a/gcc/ChangeLog b/gcc/ChangeLog index 16ca10b945e..6947477ea53 100644 --- a/gcc/ChangeLog +++ b/gcc/ChangeLog @@ -1,3 +1,12 @@ +2016-12-24 Richard Sandiford + + * tree-core.h (tree_base): Document the meaning of public_flag + for SSA names. + * tree.h (SSA_NAME_IS_VIRTUAL_OPERAND): New macro. + (SET_SSA_NAME_VAR_OR_IDENTIFIER): Record whether the variable + is a virtual operand. + * gimple-expr.h (virtual_operand_p): Use SSA_NAME_IS_VIRTUAL_OPERAND. + 2016-12-22 Prathamesh Kulkarni * tree-pretty-print.c (dump_generic_node): Change dump format for diff --git a/gcc/gimple-expr.h b/gcc/gimple-expr.h index f2ccd29a94a..faf4c53db30 100644 --- a/gcc/gimple-expr.h +++ b/gcc/gimple-expr.h @@ -105,11 +105,7 @@ static inline bool virtual_operand_p (tree op) { if (TREE_CODE (op) == SSA_NAME) - { - op = SSA_NAME_VAR (op); - if (!op) - return false; - } + return SSA_NAME_IS_VIRTUAL_OPERAND (op); if (TREE_CODE (op) == VAR_DECL) return VAR_DECL_IS_VIRTUAL_OPERAND (op); diff --git a/gcc/tree-core.h b/gcc/tree-core.h index eec2d4f35b5..59d771c1713 100644 --- a/gcc/tree-core.h +++ b/gcc/tree-core.h @@ -1091,6 +1091,9 @@ struct GTY(()) tree_base { FALLTHROUGH_LABEL_P in LABEL_DECL + SSA_NAME_IS_VIRTUAL_OPERAND in + SSA_NAME + private_flag: TREE_PRIVATE in diff --git a/gcc/tree.h b/gcc/tree.h index e3192027575..2dec19e42e6 100644 --- a/gcc/tree.h +++ b/gcc/tree.h @@ -1665,6 +1665,11 @@ extern void protected_set_expr_location (tree, location_t); /* SSA_NAME accessors. */ +/* Whether SSA_NAME NODE is a virtual operand. This simply caches the + information in the underlying SSA_NAME_VAR for efficiency. */ +#define SSA_NAME_IS_VIRTUAL_OPERAND(NODE) \ + SSA_NAME_CHECK (NODE)->base.public_flag + /* Returns the IDENTIFIER_NODE giving the SSA name a name or NULL_TREE if there is no name associated with it. */ #define SSA_NAME_IDENTIFIER(NODE) \ @@ -1683,7 +1688,16 @@ extern void protected_set_expr_location (tree, location_t); ? NULL_TREE : (NODE)->ssa_name.var) #define SET_SSA_NAME_VAR_OR_IDENTIFIER(NODE,VAR) \ - do { SSA_NAME_CHECK (NODE)->ssa_name.var = (VAR); } while (0) + do \ + { \ + tree var_ = (VAR); \ + SSA_NAME_CHECK (NODE)->ssa_name.var = var_; \ + SSA_NAME_IS_VIRTUAL_OPERAND (NODE) \ + = (var_ \ + && TREE_CODE (var_) == VAR_DECL \ + && VAR_DECL_IS_VIRTUAL_OPERAND (var_)); \ + } \ + while (0) /* Returns the statement which defines this SSA name. */ #define SSA_NAME_DEF_STMT(NODE) SSA_NAME_CHECK (NODE)->ssa_name.def_stmt -- 2.30.2