Changes in src/ResolvExpr/CastCost.cpp [5ccc733:85855b0]

File:

• src/ResolvExpr/CastCost.cpp (modified) (4 diffs)

src/ResolvExpr/CastCost.cpp

@@ -53 +53 @@
                 void postvisit( const ast::EnumInstType * enumInst ) {
                         cost = conversionCost( enumInst, dst, srcIsLvalue, symtab, env );
-
-                        if ( !enumInst->base->isCfa ) {
-                                static ast::ptr<ast::BasicType> integer = { new ast::BasicType( ast::BasicKind::SignedInt ) };
-                                Cost intCost = costCalc( integer, dst, srcIsLvalue, symtab, env );
-                                cost = intCost < cost? intCost: cost;
-                        } else if ( enumInst->base->isTyped() ) {
-                                auto baseConversionCost = 
-                                        castCost( enumInst->base->base, dst, srcIsLvalue, symtab, env );
-                                cost = baseConversionCost < cost? baseConversionCost: cost;
-                        }
                 }
 
@@ -73 +63 @@
                                 cost = conversionCost( basicType, dst, srcIsLvalue, symtab, env );
                                 if ( Cost::unsafe < cost ) {
-                                        if ( dynamic_cast<const ast::EnumInstType *>(dst)) {
-                                                cost = Cost::unsafe;
+                                        if (auto enumInst = dynamic_cast<const ast::EnumInstType *>(dst)) {
+                                                // Always explict cast only for typed enum
+                                                if (enumInst->base->isTyped) cost = Cost::unsafe;
                                         }
                                 }
@@ -83 +74 @@
                         cost = conversionCost( zero, dst, srcIsLvalue, symtab, env );
                         if ( Cost::unsafe < cost ) {
-                                if ( dynamic_cast<const ast::EnumInstType *>(dst)) {
-                                        cost = Cost::unsafe;
+                                if (auto enumInst =  dynamic_cast<const ast::EnumInstType *>(dst)) {
+                                        if (enumInst->base->isTyped) cost = Cost::unsafe;
                                 }
                         }
@@ -92 +83 @@
                         cost = conversionCost( one, dst, srcIsLvalue, symtab, env );
                         if ( Cost::unsafe < cost ) {
-                                if ( dynamic_cast<const ast::EnumInstType *>(dst)) {
-                                        cost = Cost::unsafe;
+                                if (auto enumInst = dynamic_cast<const ast::EnumInstType *>(dst)) {
+                                        if (enumInst->base->isTyped) cost = Cost::unsafe;
                                 }
                         }