Changeset 3f3bfe5a


Ignore:
Timestamp:
Oct 11, 2019, 2:23:30 PM (5 years ago)
Author:
Andrew Beach <ajbeach@…>
Branches:
ADT, arm-eh, ast-experimental, enum, forall-pointer-decay, jacob/cs343-translation, master, new-ast, new-ast-unique-expr, pthread-emulation, qualifiedEnum
Children:
396b830
Parents:
cf32116 (diff), 70b4ea20 (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 from master to new-ast. Removing old lvalue support.

Location:
src
Files:
2 deleted
21 edited

Legend:

Unmodified
Added
Removed
  • src/AST/CVQualifiers.hpp

    rcf32116 r3f3bfe5a  
    2727                Restrict = 1 << 1,
    2828                Volatile = 1 << 2,
    29                 Lvalue   = 1 << 3,
    30                 Mutex    = 1 << 4,
    31                 Atomic   = 1 << 5,
    32                 NumQualifiers = 6
     29                Mutex    = 1 << 3,
     30                Atomic   = 1 << 4,
     31                NumQualifiers = 5
    3332        };
    3433
    3534        /// Mask for equivalence-preserving qualfiers
    36         enum { EquivQualifiers = ~(Restrict | Lvalue) };
     35        enum { EquivQualifiers = ~Restrict };
    3736
    3837        /// Underlying data for qualifiers
     
    4443                                bool is_restrict : 1;
    4544                                bool is_volatile : 1;
    46                                 bool is_lvalue   : 1;
    4745                                bool is_mutex    : 1;
    4846                                bool is_atomic   : 1;
  • src/AST/Expr.cpp

    rcf32116 r3f3bfe5a  
    1010// Created On       : Wed May 15 17:00:00 2019
    1111// Last Modified By : Andrew Beach
    12 // Created On       : Thr Jun 26 12:12:00 2019
    13 // Update Count     : 3
     12// Created On       : Fri Oct  4 15:34:00 2019
     13// Update Count     : 4
    1414//
    1515
     
    8282                        // base type
    8383                        ret->result = base;
    84                         add_qualifiers( ret->result, CV::Lvalue );
    8584                }
    8685        }
     
    131130                        // lvalue, retains all levels of reference, and gains a pointer inside the references
    132131                        Type * res = addrType( arg->result );
    133                         res->set_lvalue( false ); // result of & is never an lvalue
    134132                        result = res;
    135133                } else {
     
    138136                                        dynamic_cast< const ReferenceType * >( arg->result.get() ) ) {
    139137                                Type * res = addrType( refType->base );
    140                                 res->set_lvalue( false ); // result of & is never an lvalue
    141138                                result = res;
    142139                        } else {
     
    230227        // substitute aggregate generic parameters into member type
    231228        genericSubstitution( aggregate->result ).apply( result );
    232         // ensure lvalue and appropriate restrictions from aggregate type
    233         add_qualifiers( result, aggregate->result->qualifiers | CV::Lvalue );
     229        // ensure appropriate restrictions from aggregate type
     230        add_qualifiers( result, aggregate->result->qualifiers );
    234231}
    235232
     
    257254        assert( var );
    258255        assert( var->get_type() );
    259         auto r = shallowCopy( var->get_type() );
    260         r->qualifiers |= CV::Lvalue;
    261         result = r;
     256        result = shallowCopy( var->get_type() );
    262257}
    263258
     
    376371        assert( t && i );
    377372        result = t;
    378         add_qualifiers( result, CV::Lvalue );
    379373}
    380374
     
    397391        // like MemberExpr, TupleIndexExpr is always an lvalue
    398392        result = type->types[ index ];
    399         add_qualifiers( result, CV::Lvalue );
    400393}
    401394
  • src/AST/Type.hpp

    rcf32116 r3f3bfe5a  
    5151        bool is_volatile() const { return qualifiers.is_volatile; }
    5252        bool is_restrict() const { return qualifiers.is_restrict; }
    53         bool is_lvalue() const { return qualifiers.is_lvalue; }
    5453        bool is_mutex() const { return qualifiers.is_mutex; }
    5554        bool is_atomic() const { return qualifiers.is_atomic; }
     
    5857        Type * set_volatile( bool v ) { qualifiers.is_volatile = v; return this; }
    5958        Type * set_restrict( bool v ) { qualifiers.is_restrict = v; return this; }
    60         Type * set_lvalue( bool v ) { qualifiers.is_lvalue = v; return this; }
    6159        Type * set_mutex( bool v ) { qualifiers.is_mutex = v; return this; }
    6260        Type * set_atomic( bool v ) { qualifiers.is_atomic = v; return this; }
  • src/CodeGen/GenType.cc

    rcf32116 r3f3bfe5a  
    335335                        typeString = "_Atomic " + typeString;
    336336                } // if
    337                 if ( type->get_lvalue() && ! options.genC ) {
    338                         // when not generating C code, print lvalue for debugging.
    339                         typeString = "lvalue " + typeString;
    340                 }
    341337        }
    342338} // namespace CodeGen
  • src/GenPoly/Box.cc

    rcf32116 r3f3bfe5a  
    837837                                        deref->args.push_back( new CastExpr( new VariableExpr( param ), new PointerType( Type::Qualifiers(), arg->get_type()->clone() ) ) );
    838838                                        deref->result = arg->get_type()->clone();
    839                                         deref->result->set_lvalue( true );
    840839                                        return deref;
    841840                                } // if
  • src/GenPoly/Lvalue.cc

    rcf32116 r3f3bfe5a  
    5454                                delete ret->result;
    5555                                ret->result = base->clone();
    56                                 ret->result->set_lvalue( true );
    5756                                return ret;
    5857                        } else {
     
    167166                                ReferenceType * result = strict_dynamic_cast< ReferenceType * >( appExpr->result );
    168167                                appExpr->result = result->base->clone();
    169                                 appExpr->result->set_lvalue( true );
    170168                                if ( ! inIntrinsic ) {
    171169                                        // when not in an intrinsic function, add a cast to
     
    436434                                delete ret->result;
    437435                                ret->result = castExpr->result;
    438                                 ret->result->set_lvalue( true ); // ensure result is lvalue
     436                                assert( ret->get_lvalue() ); // ensure result is lvalue
    439437                                castExpr->env = nullptr;
    440438                                castExpr->arg = nullptr;
  • src/Makefile.in

    rcf32116 r3f3bfe5a  
    232232        SynTree/Initializer.$(OBJEXT) \
    233233        SynTree/TypeSubstitution.$(OBJEXT) SynTree/Attribute.$(OBJEXT) \
    234         SynTree/DeclReplacer.$(OBJEXT) SynTree/TopLvalue.$(OBJEXT)
     234        SynTree/DeclReplacer.$(OBJEXT)
    235235am__objects_8 = CompilationState.$(OBJEXT) $(am__objects_1) \
    236236        $(am__objects_2) Concurrency/Keywords.$(OBJEXT) \
     
    698698      SynTree/TypeSubstitution.cc \
    699699      SynTree/Attribute.cc \
    700       SynTree/DeclReplacer.cc \
    701       SynTree/TopLvalue.cc
     700      SynTree/DeclReplacer.cc
    702701
    703702
     
    10301029SynTree/DeclReplacer.$(OBJEXT): SynTree/$(am__dirstamp) \
    10311030        SynTree/$(DEPDIR)/$(am__dirstamp)
    1032 SynTree/TopLvalue.$(OBJEXT): SynTree/$(am__dirstamp) \
    1033         SynTree/$(DEPDIR)/$(am__dirstamp)
    10341031Tuples/$(am__dirstamp):
    10351032        @$(MKDIR_P) Tuples
     
    13431340@AMDEP_TRUE@@am__include@ @am__quote@SynTree/$(DEPDIR)/ReferenceType.Po@am__quote@
    13441341@AMDEP_TRUE@@am__include@ @am__quote@SynTree/$(DEPDIR)/Statement.Po@am__quote@
    1345 @AMDEP_TRUE@@am__include@ @am__quote@SynTree/$(DEPDIR)/TopLvalue.Po@am__quote@
    13461342@AMDEP_TRUE@@am__include@ @am__quote@SynTree/$(DEPDIR)/TupleExpr.Po@am__quote@
    13471343@AMDEP_TRUE@@am__include@ @am__quote@SynTree/$(DEPDIR)/TupleType.Po@am__quote@
  • src/ResolvExpr/ResolveAssertions.cc

    rcf32116 r3f3bfe5a  
    156156                        for ( const auto& assn : x.assns ) {
    157157                                // compute conversion cost from satisfying decl to assertion
    158                                 assert( !assn.match.adjType->get_lvalue() );
    159158                                k += computeConversionCost(
    160159                                        assn.match.adjType, assn.decl->get_type(), false, indexer, x.env );
  • src/SymTab/Autogen.h

    rcf32116 r3f3bfe5a  
    9898                        //   type->get_qualifiers() = Type::Qualifiers();
    9999                        Type * castType = addCast->clone();
    100                         castType->get_qualifiers() -= Type::Qualifiers( Type::Lvalue | Type::Const | Type::Volatile | Type::Restrict | Type::Atomic );
     100                        castType->get_qualifiers() -= Type::Qualifiers( Type::Const | Type::Volatile | Type::Restrict | Type::Atomic );
    101101                        // castType->set_lvalue( true ); // xxx - might not need this
    102102                        dstParam = new CastExpr( dstParam, new ReferenceType( Type::Qualifiers(), castType ) );
  • src/SymTab/ManglerCommon.cc

    rcf32116 r3f3bfe5a  
    8888                                { Type::Atomic, "DA" }, // A is array, so need something unique for atmoic. For now, go with multiletter DA
    8989                                { Type::Mutex, "X" },
    90                                 { Type::Lvalue, "L" },
    9190                        };
    9291
  • src/SymTab/Validate.cc

    rcf32116 r3f3bfe5a  
    8181#include "SynTree/Label.h"             // for operator==, Label
    8282#include "SynTree/Mutator.h"           // for Mutator
    83 #include "SynTree/TopLvalue.h"         // for assertTopLvalue, clearInnerLvalue
    8483#include "SynTree/Type.h"              // for Type, TypeInstType, EnumInstType
    8584#include "SynTree/TypeSubstitution.h"  // for TypeSubstitution
     
    309308                PassVisitor<FixQualifiedTypes> fixQual;
    310309
    311                 assertTopLvalue( translationUnit );
    312310                {
    313311                        Stats::Heap::newPass("validate-A");
     
    318316                        acceptAll( translationUnit, epc ); // must happen before VerifyCtorDtorAssign, because void return objects should not exist; before LinkReferenceToTypes_old because it is an indexer and needs correct types for mangling
    319317                }
    320                 assertTopLvalue( translationUnit );
    321318                {
    322319                        Stats::Heap::newPass("validate-B");
    323320                        Stats::Time::BlockGuard guard("validate-B");
    324                         assertTopLvalue( translationUnit );
    325321                        Stats::Time::TimeBlock("Link Reference To Types", [&]() {
    326322                                acceptAll( translationUnit, lrt ); // must happen before autogen, because sized flag needs to propagate to generated functions
    327323                        });
    328                         clearInnerLvalue( translationUnit );
    329                         assertTopLvalue( translationUnit );
    330324                        Stats::Time::TimeBlock("Fix Qualified Types", [&]() {
    331325                                mutateAll( translationUnit, fixQual ); // must happen after LinkReferenceToTypes_old, because aggregate members are accessed
    332326                        });
    333                         assertTopLvalue( translationUnit );
    334327                        Stats::Time::TimeBlock("Hoist Structs", [&]() {
    335328                                HoistStruct::hoistStruct( translationUnit ); // must happen after EliminateTypedef, so that aggregate typedefs occur in the correct order
    336329                        });
    337                         assertTopLvalue( translationUnit );
    338330                        Stats::Time::TimeBlock("Eliminate Typedefs", [&]() {
    339331                                EliminateTypedef::eliminateTypedef( translationUnit ); //
    340332                        });
    341333                }
    342                 assertTopLvalue( translationUnit );
    343334                {
    344335                        Stats::Heap::newPass("validate-C");
     
    349340                        InitTweak::fixReturnStatements( translationUnit ); // must happen before autogen
    350341                }
    351                 assertTopLvalue( translationUnit );
    352342                {
    353343                        Stats::Heap::newPass("validate-D");
    354344                        Stats::Time::BlockGuard guard("validate-D");
    355                         assertTopLvalue( translationUnit );
    356345                        Stats::Time::TimeBlock("Apply Concurrent Keywords", [&]() {
    357346                                Concurrency::applyKeywords( translationUnit );
    358347                        });
    359                         clearInnerLvalue( translationUnit );
    360                         assertTopLvalue( translationUnit );
    361348                        Stats::Time::TimeBlock("Forall Pointer Decay", [&]() {
    362349                                acceptAll( translationUnit, fpd ); // must happen before autogenerateRoutines, after Concurrency::applyKeywords because uniqueIds must be set on declaration before resolution
    363350                        });
    364                         assertTopLvalue( translationUnit );
    365351                        Stats::Time::TimeBlock("Hoist Control Declarations", [&]() {
    366352                                ControlStruct::hoistControlDecls( translationUnit );  // hoist initialization out of for statements; must happen before autogenerateRoutines
    367353                        });
    368                         assertTopLvalue( translationUnit );
    369354                        Stats::Time::TimeBlock("Generate Autogen routines", [&]() {
    370355                                autogenerateRoutines( translationUnit ); // moved up, used to be below compoundLiteral - currently needs EnumAndPointerDecay_old
    371356                        });
    372                         clearInnerLvalue( translationUnit );
    373                 }
    374                 assertTopLvalue( translationUnit );
     357                }
    375358                {
    376359                        Stats::Heap::newPass("validate-E");
    377360                        Stats::Time::BlockGuard guard("validate-E");
    378                         assertTopLvalue( translationUnit );
    379361                        Stats::Time::TimeBlock("Implement Mutex Func", [&]() {
    380362                                Concurrency::implementMutexFuncs( translationUnit );
    381363                        });
    382                         clearInnerLvalue( translationUnit );
    383                         assertTopLvalue( translationUnit );
    384364                        Stats::Time::TimeBlock("Implement Thread Start", [&]() {
    385365                                Concurrency::implementThreadStarter( translationUnit );
    386366                        });
    387                         assertTopLvalue( translationUnit );
    388367                        Stats::Time::TimeBlock("Compound Literal", [&]() {
    389368                                mutateAll( translationUnit, compoundliteral );
    390369                        });
    391                         assertTopLvalue( translationUnit );
    392370                        Stats::Time::TimeBlock("Resolve With Expressions", [&]() {
    393371                                ResolvExpr::resolveWithExprs( translationUnit ); // must happen before FixObjectType because user-code is resolved and may contain with variables
    394372                        });
    395                         clearInnerLvalue( translationUnit );
    396                 }
    397                 assertTopLvalue( translationUnit );
     373                }
    398374                {
    399375                        Stats::Heap::newPass("validate-F");
    400376                        Stats::Time::BlockGuard guard("validate-F");
    401                         assertTopLvalue( translationUnit );
    402377                        Stats::Time::TimeBlock("Fix Object Type", [&]() {
    403378                                FixObjectType::fix( translationUnit );
    404379                        });
    405                         assertTopLvalue( translationUnit );
    406380                        Stats::Time::TimeBlock("Array Length", [&]() {
    407381                                ArrayLength::computeLength( translationUnit );
    408382                        });
    409                         clearInnerLvalue( translationUnit );
    410                         assertTopLvalue( translationUnit );
    411383                        Stats::Time::TimeBlock("Find Special Declarations", [&]() {
    412384                                Validate::findSpecialDecls( translationUnit );
    413385                        });
    414                         assertTopLvalue( translationUnit );
    415386                        Stats::Time::TimeBlock("Fix Label Address", [&]() {
    416387                                mutateAll( translationUnit, labelAddrFixer );
    417388                        });
    418                         assertTopLvalue( translationUnit );
    419389                        Stats::Time::TimeBlock("Handle Attributes", [&]() {
    420390                                Validate::handleAttributes( translationUnit );
    421391                        });
    422392                }
    423                 assertTopLvalue( translationUnit );
    424393        }
    425394
     
    13341303        void FixObjectType::previsit( ObjectDecl * objDecl ) {
    13351304                Type * new_type = ResolvExpr::resolveTypeof( objDecl->get_type(), indexer );
    1336                 new_type->get_qualifiers() -= Type::Lvalue; // even if typeof is lvalue, variable can never have lvalue-qualified type
    13371305                objDecl->set_type( new_type );
    13381306        }
     
    13401308        void FixObjectType::previsit( FunctionDecl * funcDecl ) {
    13411309                Type * new_type = ResolvExpr::resolveTypeof( funcDecl->type, indexer );
    1342                 new_type->get_qualifiers() -= Type::Lvalue; // even if typeof is lvalue, variable can never have lvalue-qualified type
    13431310                funcDecl->set_type( new_type );
    13441311        }
     
    13471314                if ( typeDecl->get_base() ) {
    13481315                        Type * new_type = ResolvExpr::resolveTypeof( typeDecl->get_base(), indexer );
    1349                         new_type->get_qualifiers() -= Type::Lvalue; // even if typeof is lvalue, variable can never have lvalue-qualified type
    13501316                        typeDecl->set_base( new_type );
    13511317                } // if
  • src/SynTree/AddressExpr.cc

    rcf32116 r3f3bfe5a  
    5353                        } // if
    5454                }
    55                 // result of & is never an lvalue
    56                 get_result()->set_lvalue( false );
    5755        }
    5856}
  • src/SynTree/ArrayType.cc

    rcf32116 r3f3bfe5a  
    2626ArrayType::ArrayType( const Type::Qualifiers &tq, Type *base, Expression *dimension, bool isVarLen, bool isStatic, const std::list< Attribute * > & attributes )
    2727        : Type( tq, attributes ), base( base ), dimension( dimension ), isVarLen( isVarLen ), isStatic( isStatic ) {
    28         base->set_lvalue( false );
    2928}
    3029
  • src/SynTree/CommaExpr.cc

    rcf32116 r3f3bfe5a  
    2323CommaExpr::CommaExpr( Expression *arg1, Expression *arg2 )
    2424                : Expression(), arg1( arg1 ), arg2( arg2 ) {
    25         // xxx - result of a comma expression is never an lvalue, so should set lvalue
    26         // to false on all result types. Actually doing this causes some strange things
    27         // to happen in later passes (particularly, Specialize, Lvalue, and Box). This needs to be looked into.
    2825        set_result( maybeClone( arg2->get_result() ) );
    29         // get_type->set_isLvalue( false );
    3026}
    3127
     
    4137bool CommaExpr::get_lvalue() const {
    4238        // This is wrong by C, but the current implementation uses it.
     39        // (ex: Specialize, Lvalue and Box)
    4340        return arg2->get_lvalue();
    4441}
  • src/SynTree/Expression.cc

    rcf32116 r3f3bfe5a  
    115115        assert( var->get_type() );
    116116        Type * type = var->get_type()->clone();
    117         type->set_lvalue( true );
    118117
    119118        // xxx - doesn't quite work yet - get different alternatives with the same cost
     
    125124        //      long long int value;
    126125        //      if ( decl->valueOf( var, value ) ) {
    127         //              type->set_lvalue( false );
     126        //              type->set_lvalue( false ); // Would have to move to get_lvalue.
    128127        //      }
    129128        // }
     
    384383        sub.apply( res );
    385384        result = res;
    386         result->set_lvalue( true );
    387385        result->get_qualifiers() |= aggregate->result->get_qualifiers();
    388386}
     
    433431                        // if references are still allowed in the AST, dereference returns a reference
    434432                        ret->set_result( new ReferenceType( Type::Qualifiers(), ret->get_result() ) );
    435                 } else {
    436                         // references have been removed, in which case dereference returns an lvalue of the base type.
    437                         ret->result->set_lvalue( true );
    438433                }
    439434        }
     
    591586CompoundLiteralExpr::CompoundLiteralExpr( Type * type, Initializer * initializer ) : initializer( initializer ) {
    592587        assert( type && initializer );
    593         type->set_lvalue( true );
    594588        set_result( type );
    595589}
  • src/SynTree/TupleExpr.cc

    rcf32116 r3f3bfe5a  
    7171        assertf( type->size() > index, "TupleIndexExpr index out of bounds: tuple size %d, requested index %d in expr %s", type->size(), index, toString( tuple ).c_str() );
    7272        set_result( (*std::next( type->get_types().begin(), index ))->clone() );
    73         // like MemberExpr, TupleIndexExpr is always an lvalue
    74         get_result()->set_lvalue( true );
    7573}
    7674
  • src/SynTree/Type.cc

    rcf32116 r3f3bfe5a  
    8585const char * Type::FuncSpecifiersNames[] = { "inline", "_Noreturn", "fortran" };
    8686const char * Type::StorageClassesNames[] = { "extern", "static", "auto", "register", "_Thread_local" };
    87 const char * Type::QualifiersNames[] = { "const", "restrict", "volatile", "lvalue", "mutex", "_Atomic" };
     87const char * Type::QualifiersNames[] = { "const", "restrict", "volatile", "mutex", "_Atomic" };
    8888
    8989Type * Type::stripDeclarator() {
  • src/SynTree/Type.h

    rcf32116 r3f3bfe5a  
    102102        }; // StorageClasses
    103103
    104         enum { Const = 1 << 0, Restrict = 1 << 1, Volatile = 1 << 2, Lvalue = 1 << 3, Mutex = 1 << 4, Atomic = 1 << 5, NumTypeQualifier = 6 };
     104        enum { Const = 1 << 0, Restrict = 1 << 1, Volatile = 1 << 2, Mutex = 1 << 3, Atomic = 1 << 4, NumTypeQualifier = 5 };
    105105        static const char * QualifiersNames[];
    106106        union Qualifiers {
    107                 enum { Mask = ~(Restrict | Lvalue) };
     107                enum { Mask = ~Restrict };
    108108                unsigned int val;
    109109                struct {
     
    111111                        bool is_restrict : 1;
    112112                        bool is_volatile : 1;
    113                         bool is_lvalue : 1;
    114113                        bool is_mutex : 1;
    115114                        bool is_atomic : 1;
     
    153152        bool get_volatile() const { return tq.is_volatile; }
    154153        bool get_restrict() const { return tq.is_restrict; }
    155         bool get_lvalue() const { return tq.is_lvalue; }
    156154        bool get_mutex() const { return tq.is_mutex; }
    157155        bool get_atomic() const { return tq.is_atomic; }
     
    159157        void set_volatile( bool newValue ) { tq.is_volatile = newValue; }
    160158        void set_restrict( bool newValue ) { tq.is_restrict = newValue; }
    161         void set_lvalue( bool newValue ) { tq.is_lvalue = newValue; }
    162159        void set_mutex( bool newValue ) { tq.is_mutex = newValue; }
    163160        void set_atomic( bool newValue ) { tq.is_atomic = newValue; }
  • src/SynTree/module.mk

    rcf32116 r3f3bfe5a  
    4949      SynTree/TypeSubstitution.cc \
    5050      SynTree/Attribute.cc \
    51       SynTree/DeclReplacer.cc \
    52       SynTree/TopLvalue.cc
     51      SynTree/DeclReplacer.cc
    5352
    5453SRC += $(SRC_SYNTREE)
  • src/Tuples/TupleExpansion.cc

    rcf32116 r3f3bfe5a  
    1010// Created On       : Mon May 18 07:44:20 2015
    1111// Last Modified By : Andrew Beach
    12 // Last Modified On : Fri Jul 19 14:39:00 2019
    13 // Update Count     : 22
     12// Last Modified On : Fri Oct  4 15:38:00 2019
     13// Update Count     : 23
    1414//
    1515
     
    304304                // produce the TupleType which aggregates the types of the exprs
    305305                std::list< Type * > types;
    306                 Type::Qualifiers qualifiers( Type::Const | Type::Volatile | Type::Restrict | Type::Lvalue | Type::Atomic | Type::Mutex );
     306                Type::Qualifiers qualifiers( Type::Const | Type::Volatile | Type::Restrict | Type::Atomic | Type::Mutex );
    307307                for ( Expression * expr : exprs ) {
    308308                        assert( expr->get_result() );
     
    323323                std::vector<ast::ptr<ast::Type>> types;
    324324                ast::CV::Qualifiers quals{
    325                         ast::CV::Const | ast::CV::Volatile | ast::CV::Restrict | ast::CV::Lvalue |
     325                        ast::CV::Const | ast::CV::Volatile | ast::CV::Restrict |
    326326                        ast::CV::Atomic | ast::CV::Mutex };
    327327
  • src/main.cc

    rcf32116 r3f3bfe5a  
    6060#include "ResolvExpr/Resolver.h"            // for resolve
    6161#include "SymTab/Validate.h"                // for validate
    62 #include "SynTree/TopLvalue.h"              // for assertTopLvalue, clearInn...
    6362#include "SynTree/Declaration.h"            // for Declaration
    6463#include "SynTree/Visitor.h"                // for acceptAll
     
    260259                Stats::Time::StopBlock();
    261260
    262                 //std::cerr << "Post-Parse Check" << std::endl;
    263                 clearInnerLvalue( translationUnit );
    264                 assertTopLvalue( translationUnit );
    265 
    266261                // add the assignment statement after the initialization of a type parameter
    267262                PASS( "Validate", SymTab::validate( translationUnit, symtabp ) );
     
    282277                } // if
    283278
    284                 assertTopLvalue( translationUnit );
    285279                PASS( "Fix Labels", ControlStruct::fixLabels( translationUnit ) );
    286                 assertTopLvalue( translationUnit );
    287280                PASS( "Fix Names", CodeGen::fixNames( translationUnit ) );
    288                 assertTopLvalue( translationUnit );
    289281                PASS( "Gen Init", InitTweak::genInit( translationUnit ) );
    290                 assertTopLvalue( translationUnit );
    291282                PASS( "Expand Member Tuples" , Tuples::expandMemberTuples( translationUnit ) );
    292                 assertTopLvalue( translationUnit );
    293283                if ( libcfap ) {
    294284                        // generate the bodies of cfa library functions
     
    326316                } // if
    327317
    328                 clearInnerLvalue( translationUnit );
    329                 assertTopLvalue( translationUnit );
    330 
    331318                // fix ObjectDecl - replaces ConstructorInit nodes
    332319                PASS( "Fix Init", InitTweak::fix( translationUnit, buildingLibrary() ) );
    333                 clearInnerLvalue( translationUnit );
    334                 assertTopLvalue( translationUnit );
    335320                if ( ctorinitp ) {
    336321                        dump ( translationUnit );
     
    339324
    340325                PASS( "Expand Unique Expr", Tuples::expandUniqueExpr( translationUnit ) ); // xxx - is this the right place for this? want to expand ASAP so tha, sequent passes don't need to worry about double-visiting a unique expr - needs to go after InitTweak::fix so that copy constructed return declarations are reused
    341                 assertTopLvalue( translationUnit );
    342326
    343327                PASS( "Translate EHM" , ControlStruct::translateEHM( translationUnit ) );
    344                 assertTopLvalue( translationUnit );
    345328
    346329                PASS( "Gen Waitfor" , Concurrency::generateWaitFor( translationUnit ) );
    347                 clearInnerLvalue( translationUnit );
    348                 assertTopLvalue( translationUnit );
    349330
    350331                PASS( "Convert Specializations",  GenPoly::convertSpecializations( translationUnit ) ); // needs to happen before tuple types are expanded
    351                 clearInnerLvalue( translationUnit );
    352                 assertTopLvalue( translationUnit );
    353332
    354333                PASS( "Expand Tuples", Tuples::expandTuples( translationUnit ) ); // xxx - is this the right place for this?
    355                 assertTopLvalue( translationUnit );
    356334
    357335                if ( tuplep ) {
     
    361339
    362340                PASS( "Virtual Expand Casts", Virtual::expandCasts( translationUnit ) ); // Must come after translateEHM
    363                 assertTopLvalue( translationUnit );
    364341
    365342                PASS( "Instantiate Generics", GenPoly::instantiateGeneric( translationUnit ) );
     
    368345                        return EXIT_SUCCESS;
    369346                } // if
    370                 clearInnerLvalue( translationUnit );
    371                 assertTopLvalue( translationUnit );
     347
    372348                PASS( "Convert L-Value", GenPoly::convertLvalue( translationUnit ) );
    373                 clearInnerLvalue( translationUnit );
    374                 assertTopLvalue( translationUnit );
    375349
    376350                if ( bboxp ) {
     
    379353                } // if
    380354                PASS( "Box", GenPoly::box( translationUnit ) );
    381                 clearInnerLvalue( translationUnit );
    382                 assertTopLvalue( translationUnit );
    383355
    384356                if ( bcodegenp ) {
     
    392364
    393365                CodeTools::fillLocations( translationUnit );
    394                 assertTopLvalue( translationUnit );
    395366                PASS( "Code Gen", CodeGen::generate( translationUnit, *output, ! genproto, prettycodegenp, true, linemarks ) );
    396367
Note: See TracChangeset for help on using the changeset viewer.