This is an archive of the discontinued LLVM Phabricator instance.

[flang] Add missing call to BeginReadingRecord()
ClosedPublic

Authored by klausler on Aug 13 2021, 12:04 PM.

Details

Summary

NAMELIST input needs to start with a call to BeginReadingRecord().
Internal unit input doesn't care (so unit tests were passing), but
external unit input does need the call and will assert without it.

Diff Detail

Event Timeline

klausler created this revision.Aug 13 2021, 12:04 PM
klausler requested review of this revision.Aug 13 2021, 12:04 PM
Leporacanthicus accepted this revision.Aug 17 2021, 7:34 AM
Leporacanthicus added a subscriber: Leporacanthicus.

LGTM.

(Sorry, I thought I'd approved it yesterday, but clearly didn't click the submit button!)

This revision is now accepted and ready to land.Aug 17 2021, 7:34 AM
This revision was automatically updated to reflect the committed changes.
Herald added a project: Restricted Project. · View Herald TranscriptAug 17 2021, 11:10 AM