This is an archive of the discontinued LLVM Phabricator instance.

[flang][runtime] Initialize LHS temporary in AssignTemporary.
ClosedPublic

Authored by vzakhari on May 2 2023, 12:50 PM.

Details

Summary

If LHS is of derived type that needs initialization, then it must be
initialized before doing the assignment. Otherwise, the assignment
might behave incorrectly based on uninitialized components that are
descriptors themselves.

Diff Detail

Event Timeline

vzakhari created this revision.May 2 2023, 12:50 PM
Herald added a project: Restricted Project. · View Herald TranscriptMay 2 2023, 12:50 PM
vzakhari requested review of this revision.May 2 2023, 12:50 PM
vzakhari updated this revision to Diff 518892.May 2 2023, 3:46 PM

clang-format

jeanPerier accepted this revision.May 3 2023, 12:02 AM

Looks good, thanks

This revision is now accepted and ready to land.May 3 2023, 12:02 AM