Changeset 8e18b8e for src/ResolvExpr/ConversionCost.cc

Timestamp:

May 31, 2018, 4:24:49 PM (6 years ago)

Author:

Aaron Moss <a3moss@…>

Branches:

new-env, with_gc

Children:

0182bfa, 1d7b0a8

Parents:

75308bcc

Message:

stop eagerly copying EqvClass? on lookup

File:

• src/ResolvExpr/ConversionCost.cc (modified) (2 diffs)

src/ResolvExpr/ConversionCost.cc

@@ -43 +43 @@
         Cost conversionCost( Type *src, Type *dest, const SymTab::Indexer &indexer, const TypeEnvironment &env ) {
                 if ( TypeInstType *destAsTypeInst = dynamic_cast< TypeInstType* >( dest ) ) {
-                        EqvClass eqvClass;
-                        NamedTypeDecl *namedType;
                         PRINT( std::cerr << "type inst " << destAsTypeInst->name; )
-                        if ( env.lookup( destAsTypeInst->name, eqvClass ) ) {
-                                if ( eqvClass.type ) {
-                                        return conversionCost( src, eqvClass.type, indexer, env );
+                        if ( const EqvClass* eqvClass = env.lookup( destAsTypeInst->name ) ) {
+                                if ( eqvClass->type ) {
+                                        return conversionCost( src, eqvClass->type, indexer, env );
                                 } else {
                                         return Cost::infinity;
                                 }
-                        } else if ( ( namedType = indexer.lookupType( destAsTypeInst->name ) ) ) {
+                        } else if ( NamedTypeDecl *namedType = indexer.lookupType( destAsTypeInst->name ) ) {
                                 PRINT( std::cerr << " found" << std::endl; )
                                 TypeDecl *type = dynamic_cast< TypeDecl* >( namedType );
@@ -362 +360 @@
 
         void ConversionCost::postvisit( TypeInstType *inst ) {
-                EqvClass eqvClass;
-                NamedTypeDecl *namedType;
-                if ( env.lookup( inst->name, eqvClass ) ) {
-                        cost = costFunc( eqvClass.type, dest, indexer, env );
+                if ( const EqvClass *eqvClass = env.lookup( inst->name ) ) {
+                        cost = costFunc( eqvClass->type, dest, indexer, env );
                 } else if ( TypeInstType *destAsInst = dynamic_cast< TypeInstType* >( dest ) ) {
                         if ( inst->name == destAsInst->name ) {
                                 cost = Cost::zero;
                         }
-                } else if ( ( namedType = indexer.lookupType( inst->name ) ) ) {
+                } else if ( NamedTypeDecl *namedType = indexer.lookupType( inst->name ) ) {
                         TypeDecl *type = dynamic_cast< TypeDecl* >( namedType );
                         // all typedefs should be gone by this point