This is an archive of the discontinued LLVM Phabricator instance.

[fir] Add fir.box_tdesc conversion
ClosedPublic

Authored by clementval on Nov 15 2021, 12:28 PM.

Details

Summary

This patch adds the conversion pattern for
fir.box_tdes.

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

Co-authored-by: Eric Schweitz <eschweitz@nvidia.com>

Diff Detail

Event Timeline

clementval created this revision.Nov 15 2021, 12:28 PM
Herald added a project: Restricted Project. · View Herald TranscriptNov 15 2021, 12:28 PM
Herald added a subscriber: mehdi_amini. · View Herald Transcript
clementval requested review of this revision.Nov 15 2021, 12:28 PM
Herald added a project: Restricted Project. · View Herald TranscriptNov 15 2021, 12:28 PM

LGTM.

Is this feature complete? From docs this op seems to be for Fortran 2003 select type. Also the related gentypedesc was marked todo.

flang/lib/Optimizer/CodeGen/CodeGen.cpp
468

Nit: operation -> operations

This revision is now accepted and ready to land.Nov 16 2021, 3:56 PM

LGTM.

Is this feature complete? From docs this op seems to be for Fortran 2003 select type. Also the related gentypedesc was marked todo.

It's complete as of now. Might need rework later if things change when we implement F20xx.

This revision was automatically updated to reflect the committed changes.