Changeset 8e18b8e for src/ResolvExpr/PtrsCastable.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/PtrsCastable.cc (modified) (3 diffs)

src/ResolvExpr/PtrsCastable.cc

@@ -57 +57 @@
                                 return -1;
                         } else if ( TypeInstType *typeInst = dynamic_cast< TypeInstType* >( src ) ) {
-                                EqvClass eqvClass;
                                 if ( NamedTypeDecl *ntDecl = indexer.lookupType( typeInst->get_name() ) ) {
                                         if ( TypeDecl *tyDecl = dynamic_cast< TypeDecl* >( ntDecl ) ) {
@@ -64 +63 @@
                                                 } // if
                                         } //if
-                                } else if ( env.lookup( typeInst->get_name(), eqvClass ) ) {
-                                        if ( eqvClass.data.kind == TypeDecl::Ftype ) {
+                                } else if ( const EqvClass *eqvClass = env.lookup( typeInst->get_name() ) ) {
+                                        if ( eqvClass->data.kind == TypeDecl::Ftype ) {
                                                 return -1;
                                         } // if
@@ -79 +78 @@
         int ptrsCastable( Type *src, Type *dest, const TypeEnvironment &env, const SymTab::Indexer &indexer ) {
                 if ( TypeInstType *destAsTypeInst = dynamic_cast< TypeInstType* >( dest ) ) {
-                        EqvClass eqvClass;
-                        if ( env.lookup( destAsTypeInst->get_name(), eqvClass ) ) {
+                        if ( const EqvClass *eqvClass = env.lookup( destAsTypeInst->get_name() ) ) {
                                 // xxx - should this be ptrsCastable?
-                                return ptrsAssignable( src, eqvClass.type, env );
+                                return ptrsAssignable( src, eqvClass->type, env );
                         } // if
                 } // if