diff options
author | David Malcolm <dmalcolm@redhat.com> | 2020-03-03 16:45:56 -0500 |
---|---|---|
committer | David Malcolm <dmalcolm@redhat.com> | 2020-03-04 10:44:12 -0500 |
commit | 13e3ba14eccc0b1ccf1ba9de90443ec7e524f2a6 (patch) | |
tree | 81ddd6a03a8df0692482899443b9b9461830a259 | |
parent | 94f7d7ec6ebef49a50da777fd71db3d03ee03aa0 (diff) | |
download | gcc-13e3ba14eccc0b1ccf1ba9de90443ec7e524f2a6.zip gcc-13e3ba14eccc0b1ccf1ba9de90443ec7e524f2a6.tar.gz gcc-13e3ba14eccc0b1ccf1ba9de90443ec7e524f2a6.tar.bz2 |
analyzer: remove unused private fields
gcc/analyzer/ChangeLog:
* engine.cc (worklist::worklist): Remove unused field m_eg.
(class viz_callgraph_edge): Remove unused field m_call_sedge.
(class viz_callgraph): Remove unused field m_sg.
* exploded-graph.h (worklist::::m_eg): Remove unused field.
-rw-r--r-- | gcc/analyzer/ChangeLog | 7 | ||||
-rw-r--r-- | gcc/analyzer/engine.cc | 18 | ||||
-rw-r--r-- | gcc/analyzer/exploded-graph.h | 1 |
3 files changed, 12 insertions, 14 deletions
diff --git a/gcc/analyzer/ChangeLog b/gcc/analyzer/ChangeLog index fa0fbcd..1ec8100 100644 --- a/gcc/analyzer/ChangeLog +++ b/gcc/analyzer/ChangeLog @@ -1,3 +1,10 @@ +2020-03-04 David Malcolm <dmalcolm@redhat.com> + + * engine.cc (worklist::worklist): Remove unused field m_eg. + (class viz_callgraph_edge): Remove unused field m_call_sedge. + (class viz_callgraph): Remove unused field m_sg. + * exploded-graph.h (worklist::::m_eg): Remove unused field. + 2020-03-02 David Malcolm <dmalcolm@redhat.com> * analyzer.opt (fanalyzer-show-duplicate-count): New option. diff --git a/gcc/analyzer/engine.cc b/gcc/analyzer/engine.cc index 9c3b5ad..e411d5b 100644 --- a/gcc/analyzer/engine.cc +++ b/gcc/analyzer/engine.cc @@ -1654,8 +1654,7 @@ strongly_connected_components::strong_connect (unsigned index) /* worklist's ctor. */ worklist::worklist (const exploded_graph &eg, const analysis_plan &plan) -: m_eg (eg), - m_scc (eg.get_supergraph (), eg.get_logger ()), +: m_scc (eg.get_supergraph (), eg.get_logger ()), m_plan (plan), m_queue (key_t (*this, NULL)) { @@ -3602,10 +3601,8 @@ private: class viz_callgraph_edge : public dedge<viz_callgraph_traits> { public: - viz_callgraph_edge (viz_callgraph_node *src, viz_callgraph_node *dest, - const call_superedge *call_sedge) - : dedge<viz_callgraph_traits> (src, dest), - m_call_sedge (call_sedge) + viz_callgraph_edge (viz_callgraph_node *src, viz_callgraph_node *dest) + : dedge<viz_callgraph_traits> (src, dest) {} void dump_dot (graphviz_out *gv, const dump_args_t &) const @@ -3627,9 +3624,6 @@ public: style, color, weight, constraint); pp_printf (pp, "\"];\n"); } - -private: - const call_superedge * const m_call_sedge; }; /* Subclass of digraph representing the callgraph. */ @@ -3650,7 +3644,6 @@ public: } private: - const supergraph &m_sg; hash_map<function *, viz_callgraph_node *> m_map; }; @@ -3663,7 +3656,6 @@ class viz_callgraph_cluster : public cluster<viz_callgraph_traits> /* viz_callgraph's ctor. */ viz_callgraph::viz_callgraph (const supergraph &sg) -: m_sg (sg) { cgraph_node *node; FOR_EACH_FUNCTION_WITH_GIMPLE_BODY (node) @@ -3682,11 +3674,11 @@ viz_callgraph::viz_callgraph (const supergraph &sg) viz_callgraph_node *vcg_src = get_vcg_node_for_snode (sedge->m_src); if (vcg_src->m_fun) get_vcg_node_for_function (vcg_src->m_fun)->m_num_superedges++; - if (const call_superedge *call_sedge = sedge->dyn_cast_call_superedge ()) + if (sedge->dyn_cast_call_superedge ()) { viz_callgraph_node *vcg_dest = get_vcg_node_for_snode (sedge->m_dest); viz_callgraph_edge *vcg_edge - = new viz_callgraph_edge (vcg_src, vcg_dest, call_sedge); + = new viz_callgraph_edge (vcg_src, vcg_dest); add_edge (vcg_edge); } } diff --git a/gcc/analyzer/exploded-graph.h b/gcc/analyzer/exploded-graph.h index c3bd383..c0a520a 100644 --- a/gcc/analyzer/exploded-graph.h +++ b/gcc/analyzer/exploded-graph.h @@ -710,7 +710,6 @@ private: /* The order is important here: m_scc needs to stick around until after m_queue has finished being cleaned up (the dtor calls the ordering fns). */ - const exploded_graph &m_eg; strongly_connected_components m_scc; const analysis_plan &m_plan; |