Changeset 00ac42e for src/ResolvExpr/PtrsCastable.cc

Timestamp:

Jun 1, 2018, 2:54:55 PM (7 years ago)

Author:

Aaron Moss <a3moss@…>

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, with_gc

Children:

262bd63

Parents:

41e16b1

git-author:

Aaron Moss <a3moss@…> (05/31/18 16:24:49)

git-committer:

Aaron Moss <a3moss@…> (06/01/18 14:54:55)

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