This is an archive of the discontinued LLVM Phabricator instance.

[flang] Add lowering for ASCII character constant
ClosedPublic

Authored by clementval on Feb 4 2022, 6:27 AM.

Details

Summary

Lower character constant of KIND = 1 with the
createStringLiteral helper function.

This patch is part of the upstreaming effort from fir-dev branch.

Diff Detail

Event Timeline

clementval created this revision.Feb 4 2022, 6:27 AM
Herald added a project: Restricted Project. · View Herald TranscriptFeb 4 2022, 6:27 AM
Herald added a subscriber: mehdi_amini. · View Herald Transcript
clementval requested review of this revision.Feb 4 2022, 6:27 AM
Herald added a project: Restricted Project. · View Herald TranscriptFeb 4 2022, 6:27 AM
kiranchandramohan accepted this revision.Feb 4 2022, 8:09 AM

LGTM. Including this in D118993 would be better.

flang/lib/Lower/ConvertExpr.cpp
300

Nit: may be an assert message also.

This revision is now accepted and ready to land.Feb 4 2022, 8:09 AM
schweitz accepted this revision.Feb 4 2022, 2:07 PM
clementval updated this revision to Diff 406365.Feb 7 2022, 2:16 AM

Rebase + address review comments

This revision was automatically updated to reflect the committed changes.