Skip to content

[flang] Disable submodule_3.f08 #152

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

Merged
merged 1 commit into from
Aug 20, 2024
Merged

[flang] Disable submodule_3.f08 #152

merged 1 commit into from
Aug 20, 2024

Conversation

klausler
Copy link
Contributor

The test Fortran/gfortran/regression/submodule_3.f08 is not expected to compile because it uses basic F'08 features while being compiled by gfortran under -std=f2003. Flang-new wasn't compiling it either, but that was because it had a bug (also in gfortran) dealing with the inheritance of implicit typing rules in procedure interfaces. That bug is (or shortly will be) fixed, and the test now compiles without error, an unexpected result. Disable the test.

The test Fortran/gfortran/regression/submodule_3.f08 is not expected
to compile because it uses basic F'08 features while being compiled
by gfortran under -std=f2003.  Flang-new wasn't compiling it either,
but that was because it had a bug (also in gfortran) dealing with
the inheritance of implicit typing rules in procedure interfaces.
That bug is (or shortly will be) fixed, and the test now compiles
without error, an unexpected result.  Disable the test.
Copy link

@psteinfeld psteinfeld left a comment

Choose a reason for hiding this comment

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

Looks good to me!

Copy link
Contributor

@tarunprabhu tarunprabhu left a comment

Choose a reason for hiding this comment

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

Thanks

@klausler klausler merged commit da81c66 into llvm:main Aug 20, 2024
1 check passed
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