This is an archive of the discontinued LLVM Phabricator instance.

[OpenMP] Change `__kmpc_free_shared` to include the paired allocation size
ClosedPublic

Authored by jhuber6 on Jul 21 2021, 2:14 PM.

Details

Summary

This patch changes __kmpc_free_shared to take an additional argument
corresponding to the associated allocation's size. This makes it easier to
implement the allocator in the runtime.

Diff Detail

Event Timeline

jhuber6 created this revision.Jul 21 2021, 2:14 PM
jhuber6 requested review of this revision.Jul 21 2021, 2:14 PM
Herald added projects: Restricted Project, Restricted Project, Restricted Project. · View Herald TranscriptJul 21 2021, 2:14 PM
jdoerfert added inline comments.Jul 21 2021, 2:21 PM
llvm/test/Transforms/OpenMP/remove_globalization.ll
41

this should not be here.

jhuber6 updated this revision to Diff 360610.Jul 21 2021, 2:55 PM

Fixing free shared expecting the incorrect number of args.

This revision is now accepted and ready to land.Jul 21 2021, 3:00 PM
This revision was landed with ongoing or failed builds.Jul 21 2021, 5:56 PM
This revision was automatically updated to reflect the committed changes.