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

Fix wrong classification as StatementFunction in translation to Loki IR #327

Conversation

reuterbal
Copy link
Collaborator

Sigh....

Found with scan2m.F90, correctly removing all misclassifications now.

Copy link

Documentation for this branch can be viewed at https://sites.ecmwf.int/docs/loki/327/index.html

Copy link

codecov bot commented Jun 11, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 95.18%. Comparing base (eb793e2) to head (3526b9f).
Report is 14 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #327      +/-   ##
==========================================
+ Coverage   95.15%   95.18%   +0.02%     
==========================================
  Files         168      168              
  Lines       35454    35645     +191     
==========================================
+ Hits        33737    33929     +192     
+ Misses       1717     1716       -1     
Flag Coverage Δ
lint_rules 96.39% <ø> (+0.01%) ⬆️
loki 95.16% <100.00%> (+0.02%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@reuterbal reuterbal force-pushed the 326-fparser-frontend-array-assignments-may-wrongfully-be-interpreted-as-statementfunction branch from 6db70d3 to 3526b9f Compare June 13, 2024 09:53
@reuterbal reuterbal changed the title Fparser frontend: Fix wrong classification as StatementFunction in translation to Loki IR Fix wrong classification as StatementFunction in translation to Loki IR Jun 13, 2024
@reuterbal
Copy link
Collaborator Author

I think this is ready for another look @mlange05
As discussed offline, the type attribute is gone and the heuristic is handled differently - in the process I fixed some OFP issues.

Copy link
Collaborator

@mlange05 mlange05 left a comment

Choose a reason for hiding this comment

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

Excellent! Many thanks for grinding through the extra changes and the respective fixes. Looks great now! GTG from me. :shipit:

@mlange05 mlange05 merged commit 7959f5b into main Jun 14, 2024
12 checks passed
@mlange05 mlange05 deleted the 326-fparser-frontend-array-assignments-may-wrongfully-be-interpreted-as-statementfunction branch June 14, 2024 13:21
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.

Fparser frontend: Array assignments may wrongfully be interpreted as StatementFunction
2 participants