Changes in / [4f57930:87c5f40]

Files:

• doc/generic_types/generic_types.tex (modified) (1 diff)
• src/ResolvExpr/AlternativeFinder.cc (modified) (5 diffs)
• src/ResolvExpr/AlternativeFinder.h (modified) (1 diff)
• src/SymTab/Autogen.cc (modified) (1 diff)
• src/SynTree/Expression.cc (modified) (1 diff)

doc/generic_types/generic_types.tex

@@ -226 +226 @@
 \end{lstlisting}
 Within the block, the nested version of @<@ performs @>@ and this local version overrides the built-in @<@ so it is passed to @qsort@.
-Hence, programmers can easily form local environments, adding and modifying appropriate functions, to maximize reuse of other existing functions and types.
+Hence, programmers can easily form a local environments, adding and modifying appropriate functions, to maximize reuse of other existing functions and types.
 
 Finally, \CFA allows variable overloading:

src/ResolvExpr/AlternativeFinder.cc

@@ -211 +211 @@
         }
 
-        void AlternativeFinder::addAnonConversions( const Alternative & alt ) {
-                // adds anonymous member interpretations whenever an aggregate value type is seen.
-                Expression * expr = alt.expr->clone();
-                std::unique_ptr< Expression > manager( expr ); // RAII for expr
-                alt.env.apply( expr->get_result() );
-                if ( StructInstType *structInst = dynamic_cast< StructInstType* >( expr->get_result() ) ) {
-                        NameExpr nameExpr( "" );
-                        addAggMembers( structInst, expr, alt.cost+Cost( 0, 0, 1 ), alt.env, &nameExpr );
-                } else if ( UnionInstType *unionInst = dynamic_cast< UnionInstType* >( expr->get_result() ) ) {
-                        NameExpr nameExpr( "" );
-                        addAggMembers( unionInst, expr, alt.cost+Cost( 0, 0, 1 ), alt.env, &nameExpr );
-                } // if
-        }
+        // std::unordered_map< Expression *, UniqueExpr * > ;
 
         template< typename StructOrUnionType >
@@ -232 +220 @@
                 std::list< Declaration* > members;
                 aggInst->lookup( name, members );
-
                 for ( std::list< Declaration* >::const_iterator i = members.begin(); i != members.end(); ++i ) {
                         if ( DeclarationWithType *dwt = dynamic_cast< DeclarationWithType* >( *i ) ) {
                                 alternatives.push_back( Alternative( new MemberExpr( dwt, expr->clone() ), env, newCost ) );
                                 renameTypes( alternatives.back().expr );
-                                addAnonConversions( alternatives.back() ); // add anonymous member interpretations whenever an aggregate value type is seen as a member expression.
                         } else {
                                 assert( false );
@@ -744 +730 @@
                 if ( candidates.empty() && ! errors.isEmpty() ) { throw errors; }
 
-                // compute conversionsion costs
                 for ( AltList::iterator withFunc = candidates.begin(); withFunc != candidates.end(); ++withFunc ) {
                         Cost cvtCost = computeConversionCost( *withFunc, indexer );
@@ -766 +751 @@
                         } // if
                 } // for
-                // function may return struct or union value, in which case we need to add alternatives for implicit conversions to each of the anonymous members
-                for ( const Alternative & alt : alternatives ) {
-                        addAnonConversions( alt );
-                }
-
                 candidates.clear();
                 candidates.splice( candidates.end(), alternatives );
@@ -905 +885 @@
                         )
                         renameTypes( alternatives.back().expr );
-                        addAnonConversions( alternatives.back() ); // add anonymous member interpretations whenever an aggregate value type is seen as a name expression.
+                        if ( StructInstType *structInst = dynamic_cast< StructInstType* >( (*i)->get_type() ) ) {
+                                NameExpr nameExpr( "" );
+                                addAggMembers( structInst, &newExpr, Cost( 0, 0, 1 ), env, &nameExpr );
+                        } else if ( UnionInstType *unionInst = dynamic_cast< UnionInstType* >( (*i)->get_type() ) ) {
+                                NameExpr nameExpr( "" );
+                                addAggMembers( unionInst, &newExpr, Cost( 0, 0, 1 ), env, &nameExpr );
+                        } // if
                 } // for
         }

src/ResolvExpr/AlternativeFinder.h

@@ -78 +78 @@
                 void findSubExprs( InputIterator begin, InputIterator end, OutputIterator out );
 
-                /// Adds alternatives for anonymous members
-                void addAnonConversions( const Alternative & alt );
                 /// Adds alternatives for member expressions, given the aggregate, conversion cost for that aggregate, and name of the member
                 template< typename StructOrUnionType > void addAggMembers( StructOrUnionType *aggInst, Expression *expr, const Cost &newCost, const TypeEnvironment & env, Expression * member );

src/SymTab/Autogen.cc

@@ -498 +498 @@
                 makeUnionFieldsAssignment( srcParam, dstParam, back_inserter( funcDecl->get_statements()->get_kids() ) );
                 if ( returnVal ) {
-                        funcDecl->get_statements()->get_kids().push_back( new ReturnStmt( noLabels, new VariableExpr( srcParam ) ) );
+                        if ( isDynamicLayout ) makeUnionFieldsAssignment( srcParam, returnVal, back_inserter( funcDecl->get_statements()->get_kids() ) );
+                        else funcDecl->get_statements()->get_kids().push_back( new ReturnStmt( noLabels, new VariableExpr( srcParam ) ) );
                 }
         }

src/SynTree/Expression.cc

@@ -339 +339 @@
                         return TypeSubstitution( aggInst->get_baseParameters()->begin(), aggInst->get_baseParameters()->end(), aggInst->get_parameters().begin() );
                 } else {
-                        assertf( false, "makeSub expects struct or union type for aggregate, but got: %s", toString( t ).c_str() );
+                        assertf( false, "makeSub expects struct or union type for aggregate" );
                 }
         }