[PATCH][OBVIOUS] Remove dead code in IPA ICF.
Hi. This is one another dead code removal. Patch can bootstrap on x86_64-linux-gnu and survives regression tests. I'm going to install the patch. Thanks, Martin gcc/ChangeLog: 2019-10-31 Martin Liska * ipa-icf-gimple.c (func_checker::compare_tree_ssa_label): Remove. * ipa-icf-gimple.h: Remove declaration from compare_tree_ssa_label and compare_memory_operand. --- gcc/ipa-icf-gimple.c | 8 gcc/ipa-icf-gimple.h | 6 -- 2 files changed, 14 deletions(-) diff --git a/gcc/ipa-icf-gimple.c b/gcc/ipa-icf-gimple.c index 990ee046035..7e4ffc852a8 100644 --- a/gcc/ipa-icf-gimple.c +++ b/gcc/ipa-icf-gimple.c @@ -652,14 +652,6 @@ func_checker::compare_gimple_cond (gimple *s1, gimple *s2) return compare_operand (t1, t2); } -/* Verifies that tree labels T1 and T2 correspond in FUNC1 and FUNC2. */ - -bool -func_checker::compare_tree_ssa_label (tree t1, tree t2) -{ - return compare_operand (t1, t2); -} - /* Verifies for given GIMPLE_LABEL stmts S1 and S2 that label statements are semantically equivalent. */ diff --git a/gcc/ipa-icf-gimple.h b/gcc/ipa-icf-gimple.h index 25dff31394b..8213e4f2f46 100644 --- a/gcc/ipa-icf-gimple.h +++ b/gcc/ipa-icf-gimple.h @@ -190,12 +190,6 @@ public: /* Verification function for declaration trees T1 and T2. */ bool compare_decl (const_tree t1, const_tree t2); - /* Verifies that tree labels T1 and T2 correspond. */ - bool compare_tree_ssa_label (tree t1, tree t2); - - /* Function compare for equality given memory operands T1 and T2. */ - bool compare_memory_operand (tree t1, tree t2); - /* Function responsible for comparison of various operands T1 and T2. If these components, from functions FUNC1 and FUNC2, are equal, true is returned. */
[PATCH][OBVIOUS] Remove dead code in IPA ICF.
Hi. The patch is about a small clean up, I'm going to install that as obvious. Martin gcc/ChangeLog: 2019-06-04 Martin Liska * ipa-icf.c (INCLUDE_LIST): Remove. (sem_item_optimizer::execute): Remove call to init_wpa. * ipa-icf.h (init_wpa): Remove. --- gcc/ipa-icf.c | 4 gcc/ipa-icf.h | 9 - 2 files changed, 13 deletions(-) diff --git a/gcc/ipa-icf.c b/gcc/ipa-icf.c index dbfd3640126..6307407935f 100644 --- a/gcc/ipa-icf.c +++ b/gcc/ipa-icf.c @@ -52,7 +52,6 @@ along with GCC; see the file COPYING3. If not see */ #include "config.h" -#define INCLUDE_LIST #include "system.h" #include "coretypes.h" #include "backend.h" @@ -2543,9 +2542,6 @@ sem_item_optimizer::execute (void) fprintf (dump_file, "Dump after hash based groups\n"); dump_cong_classes (); - for (unsigned int i = 0; i < m_items.length(); i++) -m_items[i]->init_wpa (); - subdivide_classes_by_equality (true); if (dump_file) diff --git a/gcc/ipa-icf.h b/gcc/ipa-icf.h index 6b81eb38b2a..2bf0f156ef6 100644 --- a/gcc/ipa-icf.h +++ b/gcc/ipa-icf.h @@ -190,9 +190,6 @@ public: /* Dump function for debugging purpose. */ DEBUG_FUNCTION void dump (void); - /* Initialize semantic item by info reachable during LTO WPA phase. */ - virtual void init_wpa (void) = 0; - /* Semantic item initialization function. */ virtual void init (void) = 0; @@ -325,10 +322,6 @@ public: ~sem_function (); - inline virtual void init_wpa (void) - { - } - virtual void init (void); virtual bool equals_wpa (sem_item *item, hash_map &ignored_nodes); @@ -432,8 +425,6 @@ public: sem_variable (varpool_node *_node, bitmap_obstack *stack); - inline virtual void init_wpa (void) {} - /* Semantic variable initialization function. */ inline virtual void init (void) {