Changeset 543159b

Timestamp:

Aug 13, 2015, 2:16:15 PM (9 years ago)

Author:

Aaron Moss <a3moss@…>

Branches:

ADT, aaron-thesis, arm-eh, ast-experimental, cleanup-dtors, ctor, deferred_resn, demangler, enum, forall-pointer-decay, gc_noraii, jacob/cs343-translation, jenkins-sandbox, master, memory, new-ast, new-ast-unique-expr, new-env, no_list, persistent-indexer, pthread-emulation, qualifiedEnum, resolv-new, string, with_gc

Children:

43ee2a8, adcdd2f

Parents:

a3a17ba

Message:

Fixed costs for casts between pointer and integer types

File:

• src/ResolvExpr/CastCost.cc (modified) (3 diffs)

src/ResolvExpr/CastCost.cc

@@ -66 +66 @@
 
         void CastCost::visit( BasicType *basicType ) {
-                if ( dynamic_cast< PointerType* >( dest ) ) {
+                PointerType *destAsPointer = dynamic_cast< PointerType* >( dest );
+                if ( destAsPointer && basicType->isInteger() ) {
                         cost = Cost( 1, 0, 0 );
                 } else {
@@ -77 +78 @@
                         if ( pointerType->get_qualifiers() <= destAsPtr->get_qualifiers() && typesCompatibleIgnoreQualifiers( pointerType->get_base(), destAsPtr->get_base(), indexer, env ) ) {
                                 cost = Cost( 0, 0, 1 );
-                        } else if ( BasicType *destAsBasic = dynamic_cast< BasicType* >( dest ) ) {
-                                if ( destAsBasic->isInteger() ) {
-                                        cost = Cost( 1, 0, 0 );
-                                } // if
                         } else {
                                 TypeEnvironment newEnv( env );
@@ -92 +89 @@
                                 } // if
                         } // if
-                } // if
+                } else if ( BasicType *destAsBasic = dynamic_cast< BasicType* >( dest ) ) {
+                        if ( destAsBasic->isInteger() ) {
+                                cost = Cost( 1, 0, 0 );
+                        } // if
+                }
         }
 } // namespace ResolvExpr