aboutsummaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorDavid Malcolm <dmalcolm@redhat.com>2020-03-03 16:45:56 -0500
committerDavid Malcolm <dmalcolm@redhat.com>2020-03-04 10:44:12 -0500
commit13e3ba14eccc0b1ccf1ba9de90443ec7e524f2a6 (patch)
tree81ddd6a03a8df0692482899443b9b9461830a259
parent94f7d7ec6ebef49a50da777fd71db3d03ee03aa0 (diff)
downloadgcc-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/ChangeLog7
-rw-r--r--gcc/analyzer/engine.cc18
-rw-r--r--gcc/analyzer/exploded-graph.h1
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;