This is an archive of the discontinued LLVM Phabricator instance.

[flang][hlfir][NFC] catch dynamically optional args with no lowering
ClosedPublic

Authored by tblah on Jul 17 2023, 8:57 AM.

Details

Summary

These should be lowered with genOptionalValue as in D154897, but I
haven't found any cases where this code path is actually hit (flang
tests, gfortran test suite), so I don't think it would be testable.
Adding an assertion for if this code path ever becomes live.

Diff Detail

Event Timeline

tblah created this revision.Jul 17 2023, 8:57 AM
Herald added projects: Restricted Project, Restricted Project. · View Herald TranscriptJul 17 2023, 8:57 AM
tblah requested review of this revision.Jul 17 2023, 8:57 AM
vzakhari accepted this revision.Jul 17 2023, 9:05 AM

Thank you, Tom. I will see if it triggers in any test after you merge it.

This revision is now accepted and ready to land.Jul 17 2023, 9:05 AM