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
C1167 : Check for exit statments in do-concurrent #345
Conversation
Note that the copyrights on all of the test files are out of date. |
a3eeb5d
to
6bec0e2
Compare
Thanks @psteinfeld. Updated. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This looks great! Well done, and thanks.
Hi Kiran, thanks for the pull request. I'll merge this commit after the issues raised in the comments are resolved. (The NVIDIA convention has been to approve a pull request with the understanding that issues raised in comments will be addressed or further discussed.) My thought is that the person who opened the issue should resolve the issue, so please respond with more comments and commits. Thanks! |
6bec0e2
to
2a99e1e
Compare
Thanks @sscalpone. I have now pushed some changes addressing the review comments provided by @klausler , @schweitzpgi and @tskeith. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good.
Thanks for resolving all of my comments. |
Addresses flang-compiler/f18#288 Original-commit: flang-compiler/f18@2a99e1e Reviewed-on: flang-compiler/f18#345
Addresses #288