This is an archive of the discontinued LLVM Phabricator instance.

[flang] Add bind(C) to buitlin C_PTR
ClosedPublic

Authored by clementval on Aug 22 2022, 12:46 AM.

Details

Summary

D131585 Adds couple of semantics check for the components of BIND(C)
derived-type. This would raise an error when a component is of C_PTR
type. Add bind(c) to the __builtin_c_ptr type so the wrong error
is not triggered.

Diff Detail

Event Timeline

clementval created this revision.Aug 22 2022, 12:46 AM
Herald added projects: Restricted Project, Restricted Project. · View Herald TranscriptAug 22 2022, 12:46 AM
clementval requested review of this revision.Aug 22 2022, 12:46 AM
peixin accepted this revision.EditedAug 22 2022, 12:52 AM

LGTM. Checked ifort 2021 iso_c_binding.f90 and it has BIND(C) for both C_PTR and C_FUNPTR.

This revision is now accepted and ready to land.Aug 22 2022, 12:52 AM
jeanPerier accepted this revision.Aug 22 2022, 2:01 AM
This revision was automatically updated to reflect the committed changes.