This is an archive of the discontinued LLVM Phabricator instance.

[flang] Handle USE association in parse tree rewriting
ClosedPublic

Authored by klausler on Jun 9 2022, 12:15 PM.

Details

Summary

f18 was treating "f() = 1" as a statement function definition
if it could be viewed as being in the specification part and
"f" was a USE-associated function returning a data pointer.
(The non-USE-associated case is fine.) Fix to allow for "f"
to be USE associated.

Diff Detail

Event Timeline

klausler created this revision.Jun 9 2022, 12:15 PM
Herald added a project: Restricted Project. · View Herald TranscriptJun 9 2022, 12:15 PM
Herald added a subscriber: jdoerfert. · View Herald Transcript
klausler requested review of this revision.Jun 9 2022, 12:15 PM

Fixes the issue reported in https://github.com/llvm/llvm-project/issues/55834. The original issue in 554.roms_r also is fixed and the benchmark passes now. Changes LGTM.

This revision is now accepted and ready to land.Jun 13 2022, 2:31 PM
This revision was automatically updated to reflect the committed changes.