Changes in / [704c9dd:7e23d0a]

Location:

src/GenPoly

Files:

• Box.cc (modified) (7 diffs)
• GenPoly.cc (modified) (2 diffs)
• GenPoly.h (modified) (2 diffs)

src/GenPoly/Box.cc

@@ -9 +9 @@
 // Author           : Richard C. Bilson
 // Created On       : Mon May 18 07:44:20 2015
-// Last Modified By : Rob Schluntz
-// Last Modified On : Tue Nov 24 15:59:33 2015
-// Update Count     : 169
+// Last Modified By : Peter A. Buhr
+// Last Modified On : Thu Nov 19 17:40:51 2015
+// Update Count     : 133
 //
 
@@ -142 +142 @@
 
         namespace {
-                std::string makePolyMonoSuffix( FunctionType * function, const TyVarMap &tyVars ) {
-                        std::stringstream name;
-
-                        // if the return type or a parameter type involved polymorphic types, then the adapter will need 
-                        // to take those polymorphic types as pointers. Therefore, there can be two different functions 
-                        // with the same mangled name, so we need to further mangle the names.
-                        if ( isPolyRet( function, tyVars ) ) {
-                                name << "P";
-                        } else {
-                                name << "M";
-                        }
-                        name << "_";
-                        std::list< DeclarationWithType *> &paramList = function->get_parameters();
-                        for ( std::list< DeclarationWithType *>::iterator arg = paramList.begin(); arg != paramList.end(); ++arg ) {
-                                if ( isPolyObj( (*arg)->get_type(), tyVars ) ) {
-                                        name << "P";
-                                } else {
-                                        name << "M";                            
-                                }
-                        } // for
-                        return name.str();
-                }
-
-                std::string mangleAdapterName( FunctionType * function, const TyVarMap &tyVars ) {
-                        return SymTab::Mangler::mangle( function ) + makePolyMonoSuffix( function, tyVars );
-                }
-
                 std::string makeAdapterName( const std::string &mangleName ) {
                         return "_adapter" + mangleName;
@@ -244 +217 @@
                                 AdapterMap & adapters = Pass1::adapters.top();
                                 for ( std::list< FunctionType *>::iterator funType = functions.begin(); funType != functions.end(); ++funType ) {
-                                        std::string mangleName = mangleAdapterName( *funType, scopeTyVars );
+                                        std::string mangleName = SymTab::Mangler::mangle( *funType );
+                                        if ( isPolyRet( *funType, scopeTyVars ) ) {
+                                                // if the return type involved polymorphic types, then the adapter will need to take those
+                                                // polymorphic types as pointers. Therefore, there can be two different functions with the same
+                                                // mangled name, so we need the mangled names to be different.
+                                                mangleName += "polyret_";
+                                        } // if
                                         if ( adapters.find( mangleName ) == adapters.end() ) {
                                                 std::string adapterName = makeAdapterName( mangleName );
@@ -368 +347 @@
                                 ret = addRetParam( appExpr, function, function->get_returnVals().front()->get_type(), arg );
                         } // if
-                        std::string mangleName = mangleAdapterName( function, tyVars );
+                        std::string mangleName = SymTab::Mangler::mangle( function );
+                        if ( isPolyRet( function, tyVars ) ) {
+                                mangleName += "polyret_";
+                        } // if
                         std::string adapterName = makeAdapterName( mangleName );
 
@@ -583 +565 @@
                                         assert( env );
                                         env->apply( realFunction );
-                                        mangleName = SymTab::Mangler::mangle( realFunction ); 
-                                        mangleName += makePolyMonoSuffix( originalFunction, exprTyVars );
+                                        mangleName = SymTab::Mangler::mangle( realFunction );
+
+                                        if ( isPolyRet( originalFunction, exprTyVars ) ) {
+                                                mangleName += "polyret_";
+                                        } // if
 
                                         AdapterMap & adapters = Pass1::adapters.top();
@@ -915 +900 @@
 
                 void Pass1::doBeginScope() {
-                        // push a copy of the current map
+                        // actually, maybe this could (should?) push
+                        // a copy of the current map
                         adapters.push(adapters.top());
                 }
@@ -937 +923 @@
                         std::set< std::string > adaptersDone;
                         for ( std::list< FunctionType *>::iterator funType = functions.begin(); funType != functions.end(); ++funType ) {
-                                std::string mangleName = mangleAdapterName( *funType, scopeTyVars );
+                                std::string mangleName = SymTab::Mangler::mangle( *funType );
+                                if ( isPolyRet( *funType, scopeTyVars ) ) {
+                                        mangleName += "polyret_";
+                                } // if
                                 if ( adaptersDone.find( mangleName ) == adaptersDone.end() ) {
                                         std::string adapterName = makeAdapterName( mangleName );

src/GenPoly/GenPoly.cc

@@ -9 +9 @@
 // Author           : Richard C. Bilson
 // Created On       : Mon May 18 07:44:20 2015
-// Last Modified By : Rob Schluntz
-// Last Modified On : Tue Nov 24 15:23:08 2015
-// Update Count     : 11
+// Last Modified By : Peter A. Buhr
+// Last Modified On : Thu Nov 19 17:23:44 2015
+// Update Count     : 10
 //
 
@@ -75 +75 @@
         }
 
-        bool isPolyObj( Type *type, const TyVarMap &tyVars ) {
-                if ( isPolyVal( type, tyVars ) ) {
-                        return true;
-                } else if ( PointerType *pt = dynamic_cast<PointerType*>( type ) ) {
-                        return isPolyObj( pt->get_base(), tyVars );
-                } else {
-                        return false;
-                }
-        }
-
         void printTyVarMap( std::ostream &os, const TyVarMap &tyVarMap ) {
                 for ( TyVarMap::const_iterator i = tyVarMap.begin(); i != tyVarMap.end(); ++i ) {

src/GenPoly/GenPoly.h

@@ -9 +9 @@
 // Author           : Richard C. Bilson
 // Created On       : Mon May 18 07:44:20 2015
-// Last Modified By : Rob Schluntz
-// Last Modified On : Tue Nov 24 15:24:38 2015
-// Update Count     : 6
+// Last Modified By : Peter A. Buhr
+// Last Modified On : Thu Nov 19 17:24:03 2015
+// Update Count     : 4
 //
 
@@ -31 +31 @@
 //      bool isPolyFun( FunctionType *fun, const TyVarMap &tyVars );
         bool isPolyVal( Type *type, const TyVarMap &tyVars );
-
-  // true if type variable or any number of pointers to type variable 
-  bool isPolyObj( Type *type, const TyVarMap &tyVars );
         void printTyVarMap( std::ostream &os, const TyVarMap &tyVarMap );
 } // namespace GenPoly