Changes in src/CodeTools/DeclStats.cc [131dbb3:eed5e48]

File:

• src/CodeTools/DeclStats.cc (modified) (11 diffs)

src/CodeTools/DeclStats.cc

@@ -63 +63 @@
                         std::map<unsigned, unsigned> p_poly;   ///< Count of decls with each percentage of polymorphic elements
                         VectorMap<unsigned> n_types;           ///< Count of decls with each number of distinct types in the pack
-                        /// Count of decls with each percentage of new types in lists.
-                        /// Types used in the parameter list that recur in the return list are not considered to be new.
-                        std::map<unsigned, unsigned> p_new;
 
                         ArgPackStats& operator+= (const ArgPackStats& o) {
@@ -74 +71 @@
                                 sum(p_poly, o.p_poly);
                                 sum(n_types, o.n_types);
-                                sum(p_new, o.p_new);
                                 
                                 return *this;
@@ -90 +86 @@
                         /// Count of uses of each non-basic type
                         std::unordered_map<std::string, unsigned> compound_type_names;
-                        /// Count of decls using each basic type
-                        std::unordered_map<std::string, unsigned> basic_type_decls;
-                        /// Count of decls using each compound type
-                        std::unordered_map<std::string, unsigned> compound_type_decls;
                         /// Stats for the parameter list
                         ArgPackStats params;
@@ -106 +98 @@
                         ArgPackStats assn_returns;
                         
-                        Stats() : n_decls(0), n_type_params(), by_name(), basic_type_names(), compound_type_names(), basic_type_decls(), compound_type_decls(), params(), returns(), n_assns(), assn_params(), assn_returns() {}
+                        Stats() : n_decls(0), n_type_params(), by_name(), basic_type_names(), compound_type_names(), params(), returns(), n_assns(), assn_params(), assn_returns() {}
 
                 public:
@@ -115 +107 @@
                                 sum( basic_type_names, o.basic_type_names );
                                 sum( compound_type_names, o.compound_type_names );
-                                sum( basic_type_decls, o.basic_type_decls );
-                                sum( compound_type_decls, o.compound_type_decls );
                                 sum( params, o.params );
                                 sum( returns, o.returns );
@@ -132 +122 @@
 
                 /// Update arg pack stats based on a declaration list
-                void analyze( Stats& stats, std::unordered_set<std::string>& seen, ArgPackStats& pstats, std::list<DeclarationWithType*>& decls ) {
+                void analyze( Stats& stats, ArgPackStats& pstats, std::list<DeclarationWithType*>& decls ) {
                         std::unordered_set<std::string> types;
-                        unsigned n = 0;        ///< number of args/returns
-                        unsigned n_basic = 0;  ///< number of basic types
-                        unsigned n_poly = 0;   ///< number of polymorphic types
-                        unsigned n_new = 0;    ///< number of new types
+                        unsigned n = 0;
+                        unsigned n_basic = 0;
+                        unsigned n_poly = 0;
                         for ( auto decl : decls ) {
                                 Type* dt = decl->get_type();
@@ -146 +135 @@
                                 dt->print( ss );
                                 types.insert( ss.str() );
-                                bool this_new = seen.insert( ss.str() ).second;
-                                if ( this_new ) { ++n_new; }
 
                                 if ( dynamic_cast<BasicType*>( dt ) ) {
                                         ++n_basic;
                                         ++stats.basic_type_names[ ss.str() ];
-                                        if ( this_new ) {
-                                                ++stats.basic_type_decls[ ss.str() ];
-                                        }
                                 } else if ( GenPoly::hasPolyBase( dt ) ) {
                                         ++n_poly;
                                 } else {
                                         ++stats.compound_type_names[ ss.str() ];
-                                        if ( this_new ) {
-                                                ++stats.compound_type_decls[ ss.str() ];
-                                        }
                                 }
                         }
@@ -170 +151 @@
                                 ++pstats.p_basic[ n_basic*100/n ];
                                 ++pstats.p_poly[ n_poly*100/n ];
-                                ++pstats.p_new[ n_new*100/n ];
                         }
                         ++pstats.n_types.at( types.size() );
@@ -176 +156 @@
                 
                 void analyzeFunc( FunctionType* fnTy, Stats& stats, ArgPackStats& params, ArgPackStats& returns ) {
-                        std::unordered_set<std::string> seen;
-                        analyze( stats, seen, params, fnTy->get_parameters() );
-                        analyze( stats, seen, returns, fnTy->get_returnVals() );
+                        analyze( stats, params, fnTy->get_parameters() );
+                        analyze( stats, returns, fnTy->get_returnVals() );
                 }
 
@@ -296 +275 @@
                         printAllMap("%_poly_" + name, [&extract](const Stats& stats) { return extract(stats).p_poly; });
                         printAllMap("n_distinct_types_" + name, [&extract](const Stats& stats) { return extract(stats).n_types; });
-                        printAllMap("%_new_types_in_" + name, [&extract](const Stats& stats) { return extract(stats).p_new; });
                 }
                 
@@ -313 +291 @@
                         printAll("basic_type_names", [](const Stats& stats) { return stats.basic_type_names.size(); });
                         printAllSparseHisto("basic_type_uses", [](const Stats& stats) { return stats.basic_type_names; });
-                        printAllSparseHisto("decls_using_basic_type", [](const Stats& stats) { return stats.basic_type_decls; });
                         printAll("compound_type_names", [](const Stats& stats) { return stats.compound_type_names.size(); });
                         printAllSparseHisto("compound_type_uses", [](const Stats& stats) { return stats.compound_type_names; });
-                        printAllSparseHisto("decls_using_compound_type", [](const Stats& stats) { return stats.compound_type_decls; });
                         printAllPack("params", [](const Stats& stats) { return stats.params; });
                         printAllPack("returns", [](const Stats& stats) { return stats.returns; });