This is an archive of the discontinued LLVM Phabricator instance.

[fir] Add fir.global_len conversion placeholder
ClosedPublic

Authored by clementval on Nov 15 2021, 5:34 AM.

Details

Summary

As for D113662, this patch just add a place holder for
the fir.global_len operation conversion. This operation
is part of F20xx and is not implemented yet.

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

Diff Detail

Event Timeline

clementval created this revision.Nov 15 2021, 5:34 AM
Herald added a project: Restricted Project. · View Herald TranscriptNov 15 2021, 5:34 AM
Herald added a subscriber: mehdi_amini. · View Herald Transcript
clementval requested review of this revision.Nov 15 2021, 5:34 AM
Herald added a project: Restricted Project. · View Herald TranscriptNov 15 2021, 5:34 AM
awarzynski accepted this revision.Nov 15 2021, 6:02 AM

Makes sense, thank you!

flang/lib/Optimizer/CodeGen/CodeGen.cpp
648

DELETEME

This revision is now accepted and ready to land.Nov 15 2021, 6:02 AM
clementval added inline comments.Nov 15 2021, 6:06 AM
flang/lib/Optimizer/CodeGen/CodeGen.cpp
648

Thanks for catching this.

flang/lib/Optimizer/CodeGen/CodeGen.cpp
648

I think the select_type has a return failure which needs deletion.

clementval marked an inline comment as done.Nov 15 2021, 7:19 AM
clementval added inline comments.
flang/lib/Optimizer/CodeGen/CodeGen.cpp
648

You are right @kiranchandramohan. I'll remove it.

clementval marked an inline comment as done.

Remove extra return + rebase

This revision was automatically updated to reflect the committed changes.