This is an archive of the discontinued LLVM Phabricator instance.

[flang] Convert assertion to a TODO
ClosedPublic

Authored by clementval on Jun 30 2022, 1:51 AM.

Details

Summary

The original assertion is not necessarily correct since the shape
argument may involve a slice of an array (an expression) and not a whole
vector with constant length. In the presence of a slice operation, the
size must be computed (left as a TODO for now).

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.Jun 30 2022, 1:51 AM
Herald added projects: Restricted Project, Restricted Project. · View Herald TranscriptJun 30 2022, 1:51 AM
Herald added a subscriber: mehdi_amini. · View Herald Transcript
clementval requested review of this revision.Jun 30 2022, 1:51 AM
jeanPerier accepted this revision.Jun 30 2022, 4:51 AM
This revision is now accepted and ready to land.Jun 30 2022, 4:51 AM
This revision was automatically updated to reflect the committed changes.