Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Adding property terminalStatementLabel to a FASTFortranDoStatement #215

Open
lsafina opened this issue Mar 13, 2024 · 4 comments
Open

Adding property terminalStatementLabel to a FASTFortranDoStatement #215

lsafina opened this issue Mar 13, 2024 · 4 comments

Comments

@lsafina
Copy link
Contributor

lsafina commented Mar 13, 2024

Design of the do statement uses FASTFortranLabelReference to represent s in DO s [,] loop-control.

However, s here is not a label reference of a do statement, but a label reference of its terminal statement.

Thus I propose to add property terminalStatementLabel -> FASTFortranIntegerLiteral to a FASTFortranDoStatement

@lsafina
Copy link
Contributor Author

lsafina commented Mar 13, 2024

@NicolasAnquetil
Copy link
Contributor

Yes FASTFortranLabelReference is not a very good name, but I would also like to reuse common concept whenever possible.
Introducing a terminalStatementLabel means that the labels of the Do statement are different from other labels and it does not seem to me that this is very much the case.

So I would like a common concept of LabelDefinition or LabelDeclaration : a label to which a Goto statement (or arithmetic if, or continue) can refer to

@NicolasAnquetil
Copy link
Contributor

inviting @uNouss to the discussion

@lsafina
Copy link
Contributor Author

lsafina commented Mar 25, 2024

ignore for the moment

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants