Skip to content

Commit

Permalink
Ast Dump Dot
Browse files Browse the repository at this point in the history
* fixed missing expression operator dumping
  - related to ref sealangdotorg/sea#19
  - related to ref sealangdotorg/sea#20
  • Loading branch information
ppaulweber committed Apr 6, 2017
1 parent 2ec2aae commit 6bcbf7f
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions src/transform/AstDumpDotPass.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -224,14 +224,14 @@ void AstDumpDotVisitor::visit( IndirectCallExpression& node )
void AstDumpDotVisitor::visit( UnaryExpression& node )
{
DotLink link( this, &node );
dumpNode( node, "Expression" ); // TODO dump operator
dumpNode( node, "Expression\n" + libcasm_ir::Value::token( node.op() ) );
RecursiveVisitor::visit( node );
}

void AstDumpDotVisitor::visit( BinaryExpression& node )
{
DotLink link( this, &node );
dumpNode( node, "Expression" ); // TODO dump operator
dumpNode( node, "Expression\n" + libcasm_ir::Value::token( node.op() ) );
RecursiveVisitor::visit( node );
}

Expand Down

4 comments on commit 6bcbf7f

@emmanuel099
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Wrong branch. Should be commited to feature/19_new_ast

@ppaulweber
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I know, but I've referenced it and due to the master merge later it doesn't matter anyway and I'll need it currently for the type inference implementation

@emmanuel099
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Other people need these changes too (without merging all the type inference changes) ;)

@ppaulweber
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

😉

Please sign in to comment.