Changeset 69911c11 for src/SymTab/Mangler.cc

Timestamp:

Dec 17, 2015, 12:00:55 PM (6 years ago)

Author:

Aaron Moss <a3moss@…>

Branches:

aaron-thesis, arm-eh, cleanup-dtors, ctor, deferred_resn, demangler, gc_noraii, jacob/cs343-translation, jenkins-sandbox, master, memory, new-ast, new-ast-unique-expr, new-env, no_list, persistent-indexer, resolv-new, string, with_gc

Children:

c23f807

Parents:

7754cde

Message:

Restore Mangler to closer to previous behaviour

File:

• src/SymTab/Mangler.cc (modified) (6 diffs)

src/SymTab/Mangler.cc

@@ -30 +30 @@
 
 namespace SymTab {
-        Mangler::Mangler( bool mangleOverridable, bool includeQualifiers )
-                : nextVarNum( 0 ), isTopLevel( true ), mangleOverridable( mangleOverridable ), includeQualifiers(includeQualifiers) {}
+        std::string Mangler::mangleType( Type *ty ) {
+                Mangler mangler( false, true );
+                maybeAccept( ty, mangler );
+                return mangler.get_mangleName();
+        }
+        
+        Mangler::Mangler( bool mangleOverridable, bool typeMode )
+                : nextVarNum( 0 ), isTopLevel( true ), mangleOverridable( mangleOverridable ), typeMode( typeMode ) {}
                 
         Mangler::Mangler( const Mangler &rhs ) : mangleName() {
@@ -38 +44 @@
                 isTopLevel = rhs.isTopLevel;
                 mangleOverridable = rhs.mangleOverridable;
-                includeQualifiers = rhs.includeQualifiers;
+                typeMode = rhs.typeMode;
         }
 
@@ -164 +170 @@
                 if ( ! params.empty() ) {
                         mangleName << "_";
-                        std::list< Type* > paramTypes;
                         for ( std::list< Expression* >::const_iterator param = params.begin(); param != params.end(); ++param ) {
                                 TypeExpr *paramType = dynamic_cast< TypeExpr* >( *param );
                                 assert(paramType && "Aggregate parameters should be type expressions");
-                                paramTypes.push_back( paramType->get_type() );
+                                maybeAccept( paramType->get_type(), *this );
                         }
-                        acceptAll( paramTypes, *this );
                         mangleName << "_";
                 }
@@ -179 +183 @@
 
         void Mangler::visit( StructInstType *aggregateUseType ) {
-                mangleGenericRef( aggregateUseType, "s" );
+                if ( typeMode ) mangleGenericRef( aggregateUseType, "s" );
+                else mangleRef( aggregateUseType, "s" );
         }
 
         void Mangler::visit( UnionInstType *aggregateUseType ) {
-                mangleGenericRef( aggregateUseType, "u" );
+                if ( typeMode ) mangleGenericRef( aggregateUseType, "u" );
+                else mangleRef( aggregateUseType, "u" );
         }
 
@@ -233 +239 @@
         void Mangler::printQualifiers( Type *type ) {
                 // skip if not including qualifiers
-                if ( ! includeQualifiers ) return;
+                if ( typeMode ) return;
                 
                 if ( ! type->get_forall().empty() ) {
@@ -253 +259 @@
                                 varNums[ (*i )->get_name() ] = std::pair< int, int >( nextVarNum++, (int )(*i )->get_kind() );
                                 for ( std::list< DeclarationWithType* >::iterator assert = (*i )->get_assertions().begin(); assert != (*i )->get_assertions().end(); ++assert ) {
-                                        Mangler sub_mangler( mangleOverridable, includeQualifiers );
+                                        Mangler sub_mangler( mangleOverridable, typeMode );
                                         sub_mangler.nextVarNum = nextVarNum;
                                         sub_mangler.isTopLevel = false;