This is an archive of the discontinued LLVM Phabricator instance.

[flang] Introduce BaseBoxType
ClosedPublic

Authored by clementval on Sep 30 2022, 6:31 AM.

Details

Summary

Introduce a BaseBoxType to be used by BoxType and
the a new ClassType that is introduced in a follow up patch.

This patch is part of the implementation of the poltymorphic
entities.
https://github.com/llvm/llvm-project/blob/main/flang/docs/PolymorphicEntities.md

Diff Detail

Event Timeline

clementval created this revision.Sep 30 2022, 6:31 AM
Herald added projects: Restricted Project, Restricted Project. · View Herald TranscriptSep 30 2022, 6:31 AM
Herald added a subscriber: mehdi_amini. · View Herald Transcript
clementval requested review of this revision.Sep 30 2022, 6:31 AM
PeteSteinfeld accepted this revision.Sep 30 2022, 10:07 AM

Aside from the two clang-format issues, all builds and tests correctly and looks good.

flang/include/flang/Optimizer/Dialect/FIRType.h
312

On my system, clang-format objects to this blank line.

flang/lib/Optimizer/Dialect/FIRType.cpp
933–934

On my system, clang-format puts all of this on a single line.

This revision is now accepted and ready to land.Sep 30 2022, 10:07 AM
This revision was landed with ongoing or failed builds.Oct 2 2022, 11:09 AM
This revision was automatically updated to reflect the committed changes.
clementval marked 2 inline comments as done.