This is an archive of the discontinued LLVM Phabricator instance.

[fir] Remove `fir.unbox` operation
ClosedPublic

Authored by clementval on Nov 10 2021, 10:00 AM.

Details

Summary

fir.unbox operation is an old operation that is no longer required.
There are couple of other operations that can be used to extract
information from a fir.box such as fir.box_rank, fir.box_addr,
fir.box_dims.

This was found during the upstreaming process.

Diff Detail

Event Timeline

clementval created this revision.Nov 10 2021, 10:00 AM
Herald added a project: Restricted Project. · View Herald TranscriptNov 10 2021, 10:00 AM
clementval requested review of this revision.Nov 10 2021, 10:00 AM
Herald added a project: Restricted Project. · View Herald TranscriptNov 10 2021, 10:00 AM
awarzynski accepted this revision.Nov 10 2021, 10:52 AM

LGTM, always nice to see some unused code being removed!

This revision is now accepted and ready to land.Nov 10 2021, 10:52 AM
This revision was automatically updated to reflect the committed changes.