Changeset d3e4d6c for src/Tuples/Explode.cc

Timestamp:

Aug 23, 2017, 6:22:07 PM (8 years ago)

Author:

Peter A. Buhr <pabuhr@…>

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:

87e08e24, cb811ac

Parents:

9f07232 (diff), bd37119 (diff)
Note: this is a merge changeset, the changes displayed below correspond to the merge itself.
Use the (diff) links above to see all the changes relative to each parent.

Message:

Merge branch 'master' of plg2:software/cfa/cfa-cc

File:

• src/Tuples/Explode.cc (modified) (2 diffs)

src/Tuples/Explode.cc

@@ -15 +15 @@
 
 #include "Explode.h"
+#include <list>                  // for list
 
-#include <list>               // for list
-
-#include "SynTree/Mutator.h"  // for Mutator
+#include "SynTree/Mutator.h"     // for Mutator
+#include "Common/PassVisitor.h"  // for PassVisitor
 
 namespace Tuples {
         namespace {
-                struct AddrExploder : public Mutator {
+                // remove one level of reference from a reference type -- may be useful elsewhere.
+                Type * getReferenceBase( Type * t ) {
+                        if ( ReferenceType * refType = dynamic_cast<ReferenceType *>( t ) ) {
+                                return refType->get_base();
+                        } else {
+                                // for the moment, I want to know immediately if a non-reference type is ever passed in here.
+                                assertf( false, "getReferenceBase for non-ref: %s", toString( refType ).c_str() );
+                                return nullptr;
+                        }
+                }
+
+                struct CastExploder {
+                        bool castAdded = false;
                         bool foundUniqueExpr = false;
-                        Expression * applyAddr( Expression * expr, bool first = true ) {
+                        Expression * applyCast( Expression * expr, bool first = true ) {
                                 if ( TupleExpr * tupleExpr = dynamic_cast< TupleExpr * >( expr ) ){
                                         foundUniqueExpr = true;
                                         std::list< Expression * > exprs;
                                         for ( Expression *& expr : tupleExpr->get_exprs() ) {
-                                                // move & into tuple exprs
-                                                exprs.push_back( applyAddr( expr, false ) );
+                                                // move cast into tuple exprs
+                                                exprs.push_back( applyCast( expr, false ) );
                                         }
-                                        // want the top-level expression to be address-taken, but not nested
+                                        // want the top-level expression to be cast to reference type, but not nested
                                         // tuple expressions
                                         if ( first ) {
-                                                return new AddressExpr( new TupleExpr( exprs ) );
+                                                castAdded = true;
+                                                Expression * tupleExpr = new TupleExpr( exprs );
+                                                return new CastExpr( tupleExpr, new ReferenceType( Type::Qualifiers(), tupleExpr->result->clone() ) );
                                         } else {
                                                 return new TupleExpr( exprs );
                                         }
                                 }
-                                // anything else should be address-taken as normal
-                                return new AddressExpr( expr->clone() );
+                                if ( dynamic_cast<ReferenceType*>( expr->result ) ) {
+                                        // don't need to cast reference type to another reference type
+                                        return expr->clone();
+                                } else {
+                                        // anything else should be cast to reference as normal
+                                        castAdded = true;
+                                        return new CastExpr( expr->clone(), new ReferenceType( Type::Qualifiers(), expr->result->clone() ) );
+                                }
                         }
 
-                        virtual Expression * mutate( UniqueExpr * uniqueExpr ) {
-                                // move & into unique expr so that the unique expr has type T* rather than
+                        Expression * postmutate( UniqueExpr * uniqueExpr ) {
+                                // move cast into unique expr so that the unique expr has type T& rather than
                                 // type T. In particular, this transformation helps with generating the
-                                // correct code for address-taken member tuple expressions, since the result
-                                // should now be a tuple of addresses rather than the address of a tuple.
+                                // correct code for reference-cast member tuple expressions, since the result
+                                // should now be a tuple of references rather than a reference to a tuple.
                                 // Still, this code is a bit awkward, and could use some improvement.
-                                if ( dynamic_cast< AddressExpr * > ( uniqueExpr->get_expr() ) ) {
-                                        // this unique expression has already been mutated or otherwise shouldn't be (can't take the address-of an address-of expression)
-                                        return uniqueExpr;
+                                UniqueExpr * newUniqueExpr = new UniqueExpr( applyCast( uniqueExpr->get_expr() ), uniqueExpr->get_id() );
+                                delete uniqueExpr;
+                                if ( castAdded ) {
+                                        // if a cast was added by applyCast, then unique expr now has one more layer of reference
+                                        // than it had coming into this function. To ensure types still match correctly, need to cast
+                                        //  to reference base so that outer expressions are still correct.
+                                        castAdded = false;
+                                        Type * toType = getReferenceBase( newUniqueExpr->result );
+                                        return new CastExpr( newUniqueExpr, toType->clone() );
                                 }
-                                UniqueExpr * newUniqueExpr = new UniqueExpr( applyAddr( uniqueExpr->get_expr() ), uniqueExpr->get_id() );
-                                delete uniqueExpr;
-                                UntypedExpr * deref = UntypedExpr::createDeref( Mutator::mutate( newUniqueExpr ) );
-                                return deref;
+                                return newUniqueExpr;
                         }
 
-                        virtual Expression * mutate( TupleIndexExpr * tupleExpr ) {
+
+                        Expression * postmutate( TupleIndexExpr * tupleExpr ) {
                                 // tuple index expr needs to be rebuilt to ensure that the type of the
                                 // field is consistent with the type of the tuple expr, since the field
-                                // may have changed from type T to T*.
-                                Expression * expr = tupleExpr->get_tuple()->acceptMutator( *this );
+                                // may have changed from type T to T&.
+                                Expression * expr = tupleExpr->get_tuple();
                                 tupleExpr->set_tuple( nullptr );
                                 TupleIndexExpr * ret = new TupleIndexExpr( expr, tupleExpr->get_index() );
@@ -73 +97 @@
         } // namespace
 
-        Expression * distributeAddr( Expression * expr ) {
-                AddrExploder addrExploder;
-                expr = expr->acceptMutator( addrExploder );
-                if ( ! addrExploder.foundUniqueExpr ) {
-                        expr = new AddressExpr( expr );
+        Expression * distributeReference( Expression * expr ) {
+                PassVisitor<CastExploder> exploder;
+                expr = expr->acceptMutator( exploder );
+                if ( ! exploder.pass.foundUniqueExpr ) {
+                        // if a UniqueExpr was found, then the cast has already been added inside the UniqueExpr as appropriate
+                        expr = new CastExpr( expr, new ReferenceType( Type::Qualifiers(), expr->result->clone() ) );
                 }
                 return expr;