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

Used valid FORTRAN test program for a couple frontend tests + Made floatlit2string() convert the FORTRAN real literal strings into python floats. #1733

Merged
merged 1 commit into from
Nov 7, 2024

Conversation

pratyai
Copy link
Collaborator

@pratyai pratyai commented Nov 5, 2024

Replace some of the invalid fortran test programs with valid ones (that passes under gfortran -Wall). This breaks some of the tests, so add fix for them too. test_fortran_frontend_loop1() still has an invalid test program, but the nearest valid one breaks for a different reason, and will be fixed later.

passes under `gfortran -Wall`). This breaks some of the tests, so add
fix for them too. `test_fortran_frontend_loop1()` still has an invalid
test program, but the nearest valid one breaks for a different reason,
and will be fixed later.
@pratyai pratyai marked this pull request as ready for review November 5, 2024 23:41
@pratyai pratyai requested a review from acalotoiu November 5, 2024 23:42
Copy link
Contributor

@acalotoiu acalotoiu left a comment

Choose a reason for hiding this comment

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

LGTM

@phschaad phschaad added this pull request to the merge queue Nov 6, 2024
Merged via the queue into spcl:main with commit 5669030 Nov 7, 2024
10 checks passed
github-merge-queue bot pushed a commit that referenced this pull request Nov 10, 2024
…test program. (#1736)

A follow up of #1733 that took a bit longer to figure out.

This change breaks the test on `multi_sdfg` branch, but there the
front-end is doing something additionally wrong (it somehow promotes
`nclv` into a input parameter for the whole program) -- but it will even
take longer to figure out why.
@pratyai pratyai deleted the fix-float-parser branch November 15, 2024 11:12
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

Successfully merging this pull request may close these issues.

3 participants