From: Martin Liska Date: Mon, 2 May 2016 09:20:01 +0000 (+0200) Subject: Symbol summary: refactor usage of gcc_checking_asserts X-Git-Tag: upstream/12.2.0~47410 X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=0ec955c28f187b8799cfbf509eb6adf3b74be21c;p=platform%2Fupstream%2Fgcc.git Symbol summary: refactor usage of gcc_checking_asserts * symbol-summary.h (function_summary::function_summary): Remove checking assert for all cgraph nodes. (function_summary::get): Check summary_uid. (symtab_insertion): Check summary_uid. From-SVN: r235712 --- diff --git a/gcc/ChangeLog b/gcc/ChangeLog index fecfdab..0a48bf2 100644 --- a/gcc/ChangeLog +++ b/gcc/ChangeLog @@ -1,3 +1,10 @@ +2016-05-02 Martin Liska + + * symbol-summary.h (function_summary::function_summary): + Remove checking assert for all cgraph nodes. + (function_summary::get): Check summary_uid. + (symtab_insertion): Check summary_uid. + 2016-05-02 Claudiu Zissulescu * config/arc/arc-protos.h (compact_memory_operand_p): Declare. diff --git a/gcc/symbol-summary.h b/gcc/symbol-summary.h index c0dd5aa..a22352b 100644 --- a/gcc/symbol-summary.h +++ b/gcc/symbol-summary.h @@ -39,13 +39,6 @@ public: function_summary (symbol_table *symtab, bool ggc = false): m_ggc (ggc), m_map (13, ggc), m_insertion_enabled (true), m_symtab (symtab) { - if (flag_checking) - { - cgraph_node *node; - FOR_EACH_FUNCTION (node) - gcc_assert (node->summary_uid > 0); - } - m_symtab_insertion_hook = symtab->add_cgraph_insertion_hook (function_summary::symtab_insertion, this); @@ -124,6 +117,7 @@ public: /* Getter for summary callgraph node pointer. */ T* get (cgraph_node *node) { + gcc_checking_assert (node->summary_uid); return get (node->summary_uid); } @@ -148,6 +142,7 @@ public: /* Symbol insertion hook that is registered to symbol table. */ static void symtab_insertion (cgraph_node *node, void *data) { + gcc_checking_assert (node->summary_uid); function_summary *summary = (function_summary *) (data); if (summary->m_insertion_enabled)