Changeset 77971f6 for src/ResolvExpr/AlternativeFinder.cc

Timestamp:

Oct 27, 2016, 3:24:02 PM (7 years ago)

Author:

Rob Schluntz <rschlunt@…>

Branches:

ADT, aaron-thesis, arm-eh, ast-experimental, cleanup-dtors, deferred_resn, demangler, enum, forall-pointer-decay, jacob/cs343-translation, jenkins-sandbox, master, new-ast, new-ast-unique-expr, new-env, no_list, persistent-indexer, pthread-emulation, qualifiedEnum, resolv-new, with_gc

Children:

3f0c6a5

Parents:

a1e67dd

git-author:

Rob Schluntz <rschlunt@…> (10/27/16 15:11:00)

git-committer:

Rob Schluntz <rschlunt@…> (10/27/16 15:24:02)

Message:

resolve ctor/dtors for UniqueExprs?

File:

• src/ResolvExpr/AlternativeFinder.cc (modified) (4 diffs)

src/ResolvExpr/AlternativeFinder.cc

@@ -41 +41 @@
 #include "Common/utility.h"
 #include "InitTweak/InitTweak.h"
+#include "InitTweak/GenInit.h"
 #include "ResolveTypeof.h"
 
@@ -207 +208 @@
         }
 
+        // std::unordered_map< Expression *, UniqueExpr * > ;
+
         template< typename StructOrUnionType >
         void AlternativeFinder::addAggMembers( StructOrUnionType *aggInst, Expression *expr, const Cost &newCost, const TypeEnvironment & env, Expression * member ) {
-
                 // by this point, member must be a name expr
                 NameExpr * nameExpr = safe_dynamic_cast< NameExpr * >( member );
@@ -434 +436 @@
                 // flatten actuals so that each actual has an atomic (non-tuple) type
                 AltList exploded;
-                Tuples::explode( actuals, back_inserter( exploded ) );
+                Tuples::explode( actuals, indexer, back_inserter( exploded ) );
 
                 AltList::iterator actualExpr = exploded.begin();
@@ -1055 +1057 @@
 
         void AlternativeFinder::visit( UniqueExpr *unqExpr ) {
+                // this won't work because the unqExprs wont share an expression anymore...
                 AlternativeFinder finder( indexer, env );
                 finder.findWithAdjustment( unqExpr->get_expr() );
                 for ( Alternative & alt : finder.alternatives ) {
-                        // xxx - it's possible that this won't always do the right thing, i.e. two UniqueExprs
-                        // with the same ID may resolve to different expressions. If this ever happens, it might be necessary
-                        // to try to select an alternative here (i.e. error is there is not a unique min-cost expression).
-                        // One other thought is to to resolve each ID once and map the IDs to resolved expressions,
-                        // but this isn't as simple as it sounds since the alternative is not selected here, meaning it might
-                        // require complicated tracking throughout the system.
-
-                        // brand the new UniqueExprs with the same id so that they are recognized as the same expression by the expansion pass
-                        alternatives.push_back( Alternative( new UniqueExpr( alt.expr->clone(), unqExpr->get_id() ), env, Cost::zero ) );
+                        // xxx - attach a resolved ConstructorInit node?
+                        // xxx - is it possible to make the objDecl's type const?
+                        static UniqueName tempNamer( "_unq_expr_" );
+                        ObjectDecl * objDecl = new ObjectDecl( tempNamer.newName(), DeclarationNode::NoStorageClass, LinkageSpec::Cforall, nullptr, alt.expr->get_result()->clone(), nullptr );
+                        // must be done on two lines because genCtorInit accesses objDecl's fields
+                        objDecl->set_init( InitTweak::genCtorInit( objDecl ) );
+
+                        UniqueExpr * newUnqExpr = new UniqueExpr( alt.expr->clone(), unqExpr->get_id() );
+                        newUnqExpr->set_object( objDecl );
+
+                        resolveObject( indexer, objDecl );
+
+                        alternatives.push_back( Alternative( newUnqExpr, env, Cost::zero ) );
                 }
         }