This is an archive of the discontinued LLVM Phabricator instance.

[flang] Add semantics test for coshape() function
ClosedPublic

Authored by ngeorge1098 on Jun 30 2022, 4:25 PM.

Diff Detail

Event Timeline

ngeorge1098 created this revision.Jun 30 2022, 4:25 PM
Herald added a project: Restricted Project. · View Herald Transcript
Herald added a subscriber: jdoerfert. · View Herald Transcript
ngeorge1098 requested review of this revision.Jun 30 2022, 4:25 PM
ktras added a comment.Jul 5 2022, 11:49 AM

Test looks great, but I suggest adding to the error section by adding an attempt to assign the result of a correct coshape() call to a variable of type character, as there is no automatic conversion upon assignment between character and integer (which is of course the correct type of the returned value).

Add assignment mismatch of the result of coshape() to a variable of type character

ktras accepted this revision.Jul 5 2022, 1:49 PM

LGTM!

This revision is now accepted and ready to land.Jul 5 2022, 1:49 PM
This revision was automatically updated to reflect the committed changes.