Skip to content

Conversation

klausler
Copy link
Contributor

One test case in finalize_38.f90 expects that an allocatable component of an expression result will not be finalized. The flang-new compiler is being updated by the pull request

llvm/llvm-project#153509

to ensure finalization of allocatables when they are deallocated, since this is an event that triggers finalization (F'2023 7.5.6.3p2) on a finalizable (3.71) data entity (3.41).

norm2_3.f90
pr96711.f90

# pmk?
Copy link
Contributor

Choose a reason for hiding this comment

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

This test was not in the description?

use_rename_8.f90

# The "stop 171" case expects an incorrect number of finalizations.
finalize_38.f90
Copy link
Contributor

Choose a reason for hiding this comment

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

From the description of the PR, it sounds like this test will never pass with flang because, gfortran's finalization behavior is different from flang's. If this is the case, it may be better to move this to the UNSUPPORTED_FILES list above.

One test case in finalize_38.f90 expects that an allocatable
component of an expression result will not be finalized.  The
flang-new compiler is being updated by the pull request

  llvm/llvm-project#153509

to ensure finalization of allocatables when they are deallocated,
since this is an event that triggers finalization (F'2023 7.5.6.3p2)
on a finalizable (3.71) data entity (3.41).
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 for the change.

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