Changeset 61e362f for src/Parser

Timestamp:

Jan 23, 2024, 2:16:13 PM (14 months ago)

Author:

Andrew Beach <ajbeach@…>

Branches:

master

Children:

71b5aad5

Parents:

a4ed165

Message:

Changed notZeroExpr so that expressions with conditional contexts are handled in the resolver instead of the parser. Bugs kept the same from being done with statements. (Also a bit of clean-up from the last commit and a small fix in code-gen.)

Location:

src/Parser

Files:

• ExpressionNode.cc (modified) (2 diffs)
• parserutility.cc (modified) (1 diff)
• parserutility.h (modified) (1 diff)

src/Parser/ExpressionNode.cc

@@ -673 +673 @@
                 ast::LogicalFlag flag ) {
         return new ast::LogicalExpr( location,
-                notZeroExpr( maybeMoveBuild( expr_node1 ) ),
-                notZeroExpr( maybeMoveBuild( expr_node2 ) ),
+                maybeMoveBuild( expr_node1 ),
+                maybeMoveBuild( expr_node2 ),
                 flag
         );
@@ -713 +713 @@
                 ExpressionNode * expr_node3 ) {
         return new ast::ConditionalExpr( location,
-                notZeroExpr( maybeMoveBuild( expr_node1 ) ),
+                maybeMoveBuild( expr_node1 ),
                 maybeMoveBuild( expr_node2 ),
                 maybeMoveBuild( expr_node3 )

src/Parser/parserutility.cc

@@ -27 +27 @@
 //    if ( (int)(x != 0) ) ...
 
-ast::Expr * notZeroExpr( ast::Expr * orig ) {
+ast::Expr * notZeroExpr( const ast::Expr * orig ) {
         return ( !orig ) ? nullptr : new ast::CastExpr( orig->location,
                 ast::UntypedExpr::createCall( orig->location,

src/Parser/parserutility.h

@@ -21 +21 @@
 }
 
-ast::Expr * notZeroExpr( ast::Expr *orig );
+ast::Expr * notZeroExpr( const ast::Expr *orig );
 
 template< typename T >