Changeset 150ec33 for src/ResolvExpr/ConversionCost.cc

Timestamp:

Oct 19, 2017, 11:15:35 AM (6 years ago)

Author:

Rob Schluntz <rschlunt@…>

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

Children:

f232977

Parents:

d411426d

git-author:

Rob Schluntz <rschlunt@…> (10/13/17 16:03:35)

git-committer:

Rob Schluntz <rschlunt@…> (10/19/17 11:15:35)

Message:

Fix bug in computing union conversion costs

File:

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

src/ResolvExpr/ConversionCost.cc

@@ -306 +306 @@
                 // recursively compute conversion cost from T1 to T2.
                 // cv can be safely dropped because of 'implicit dereference' behavior.
-                refType->get_base()->accept( *this );
-                if ( refType->get_base()->get_qualifiers() == dest->get_qualifiers() ) {
+                refType->base->accept( *this );
+                if ( refType->base->get_qualifiers() == dest->get_qualifiers() ) {
                         cost.incReference();  // prefer exact qualifiers
-                } else if ( refType->get_base()->get_qualifiers() < dest->get_qualifiers() ) {
+                } else if ( refType->base->get_qualifiers() < dest->get_qualifiers() ) {
                         cost.incSafe(); // then gaining qualifiers
                 } else {
@@ -321 +321 @@
         void ConversionCost::visit(StructInstType *inst) {
                 if ( StructInstType *destAsInst = dynamic_cast< StructInstType* >( dest ) ) {
-                        if ( inst->get_name() == destAsInst->get_name() ) {
+                        if ( inst->name == destAsInst->name ) {
                                 cost = Cost::zero;
                         } // if
@@ -328 +328 @@
 
         void ConversionCost::visit(UnionInstType *inst) {
-                if ( StructInstType *destAsInst = dynamic_cast< StructInstType* >( dest ) ) {
-                        if ( inst->get_name() == destAsInst->get_name() ) {
+                if ( UnionInstType *destAsInst = dynamic_cast< UnionInstType* >( dest ) ) {
+                        if ( inst->name == destAsInst->name ) {
                                 cost = Cost::zero;
                         } // if