feature/char-support #3

2023-03-24T00:19:34+01:00

nitrix commented

2023-03-24 00:19:34 +01:00

No description provided.

nitrix added 5 commits 2023-03-24 00:19:34 +01:00

Void Type: Fix a problem where calls to void functions were ignored aef2c84fdc

The ContextAnalysis visitor for braced blocks was not updated during void support implementation which is why I do it now. Although we are re-using the functionCall production rule a different visitor is required for visiting function calls directly inside braced blocks. This is because the other place where functionCalls are used the functionCall is annotated with a label with wraps the functionCall context inside a label context. Fortunately this is in fact simple wrapping so we just implement a visitor for the wrapping class that unwraps the functionCall and passes it to the visitor that actually implements the functionCall.

GenASM+Void: Generate an implicit return 5f0e84198a

Void functions are allowed to be completely free of return statements. The problem is that this does not work on its own in assembler. We have to generate a return statement at the end of the function body if there is no return statement already.

GenASM: Set the bytesToClearFromStack variable earlier a7e93f4f01

If you set the variable after visiting the function body all children will see bytesToClearFromStack == 0.

Char: Add support for chars. 2aff9b3d0d

First try at implementing chars. There are some problems with the grammar where I was not able to separate the single quotes from the captured char, but I managed to hack it together in the ContextAnalysis.

It's currently not possible to do any 'math' with chars, but I will give it a try soon.

ContextAnalysis: Make comparing more generous 259ac49981

Previously we only allowed numeric types to be compared using >,<,<= and >=. Now we allow all primitives types to be compared using the operators mentioned above.

It is now also possible to compare and assign ints and chars to each other.

nitrix merged commit bcc9f29ff5 into master

2023-03-24 17:03:24 +01:00

nitrix referenced this issue from a commit

2023-03-24 17:03:24 +01:00

Merge pull request 'feature/char-support' (#3) from feature/char-support into master

nitrix deleted branch feature/char-support

2023-03-24 17:03:27 +01:00

Sign in to join this conversation.

1 Participants

Notifications

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: nitrix/klang#3