Changes in src/AST/SymbolTable.cpp [d859a30:bb7422a]

File:

• src/AST/SymbolTable.cpp (modified) (4 diffs)

src/AST/SymbolTable.cpp

@@ -260 +260 @@
 void SymbolTable::addId( const DeclWithType * decl, const Expr * baseExpr ) {
         // default handling of conflicts is to raise an error
-        addIdCommon( decl, OnConflict::error(), baseExpr, decl->isDeleted ? decl : nullptr );
+        addId( decl, OnConflict::error(), baseExpr, decl->isDeleted ? decl : nullptr );
 }
 
 void SymbolTable::addDeletedId( const DeclWithType * decl, const Decl * deleter ) {
         // default handling of conflicts is to raise an error
-        addIdCommon( decl, OnConflict::error(), nullptr, deleter );
+        addId( decl, OnConflict::error(), nullptr, deleter );
 }
 
@@ -677 +677 @@
 }
 
-void SymbolTable::addIdCommon(
-                const DeclWithType * decl, SymbolTable::OnConflict handleConflicts,
-                const Expr * baseExpr, const Decl * deleter ) {
+void SymbolTable::addId(
+                const DeclWithType * decl, SymbolTable::OnConflict handleConflicts, const Expr * baseExpr,
+                const Decl * deleter ) {
         SpecialFunctionKind kind = getSpecialFunctionKind(decl->name);
         if (kind == NUMBER_OF_KINDS) { // not a special decl
-                addIdToTable(decl, decl->name, idTable, handleConflicts, baseExpr, deleter);
+                addId(decl, decl->name, idTable, handleConflicts, baseExpr, deleter);
         }
         else {
@@ -695 +695 @@
                         assertf(false, "special decl with non-function type");
                 }
-                addIdToTable(decl, key, specialFunctionTable[kind], handleConflicts, baseExpr, deleter);
-        }
-}
-
-void SymbolTable::addIdToTable(
-                const DeclWithType * decl, const std::string & lookupKey,
-                IdTable::Ptr & table, SymbolTable::OnConflict handleConflicts,
-                const Expr * baseExpr, const Decl * deleter ) {
+                addId(decl, key, specialFunctionTable[kind], handleConflicts, baseExpr, deleter);
+        }
+}
+
+void SymbolTable::addId(
+                const DeclWithType * decl, const std::string & lookupKey, IdTable::Ptr & table, SymbolTable::OnConflict handleConflicts, const Expr * baseExpr,
+                const Decl * deleter ) {
         ++*stats().add_calls;
         const std::string &name = decl->name;
@@ -779 +778 @@
 void SymbolTable::addMembers(
                 const AggregateDecl * aggr, const Expr * expr, SymbolTable::OnConflict handleConflicts ) {
-        for ( const ptr<Decl> & decl : aggr->members ) {
-                auto dwt = decl.as<DeclWithType>();
-                if ( nullptr == dwt ) continue;
-                addIdCommon( dwt, handleConflicts, expr );
-                // Inline through unnamed struct/union members.
-                if ( "" != dwt->name ) continue;
-                const Type * t = dwt->get_type()->stripReferences();
-                if ( auto rty = dynamic_cast<const BaseInstType *>( t ) ) {
-                        if ( ! dynamic_cast<const StructInstType *>(rty)
-                                && ! dynamic_cast<const UnionInstType *>(rty) ) continue;
-                        ResolvExpr::Cost cost = ResolvExpr::Cost::zero;
-                        ast::ptr<ast::TypeSubstitution> tmp = expr->env;
-                        expr = mutate_field(expr, &Expr::env, nullptr);
-                        const Expr * base = ResolvExpr::referenceToRvalueConversion( expr, cost );
-                        base = mutate_field(base, &Expr::env, tmp);
-
-                        addMembers(
-                                rty->aggr(), new MemberExpr{ base->location, dwt, base }, handleConflicts );
+        for ( const Decl * decl : aggr->members ) {
+                if ( auto dwt = dynamic_cast< const DeclWithType * >( decl ) ) {
+                        addId( dwt, handleConflicts, expr );
+                        if ( dwt->name == "" ) {
+                                const Type * t = dwt->get_type()->stripReferences();
+                                if ( auto rty = dynamic_cast<const BaseInstType *>( t ) ) {
+                                        if ( ! dynamic_cast<const StructInstType *>(rty)
+                                                && ! dynamic_cast<const UnionInstType *>(rty) ) continue;
+                                        ResolvExpr::Cost cost = ResolvExpr::Cost::zero;
+                                        ast::ptr<ast::TypeSubstitution> tmp = expr->env;
+                                        expr = mutate_field(expr, &Expr::env, nullptr);
+                                        const Expr * base = ResolvExpr::referenceToRvalueConversion( expr, cost );
+                                        base = mutate_field(base, &Expr::env, tmp);
+
+                                        addMembers(
+                                                rty->aggr(), new MemberExpr{ base->location, dwt, base }, handleConflicts );
+                                }
+                        }
                 }
         }