This is an archive of the discontinued LLVM Phabricator instance.

Adding support to the OpenMP mutexinoutset dependency type
ClosedPublic

Authored by smateo on Feb 1 2019, 2:45 AM.

Details

Summary

This feature was already implemented in the LLVM OpenMP runtime, so we are only modifying CLANG in this revision.

The value used to identify a mutexinoutset dep-type in the LLVM OpenMP runtime is 4.

I had to adapt some tests because they were relying on an error message that changed with the introduction of this new dep-type. Apart from those, I also modified test/OpenMP/task_codegen.cpp to test that we emit the right code.

Diff Detail

Repository
rC Clang

Event Timeline

smateo created this revision.Feb 1 2019, 2:45 AM
Herald added a project: Restricted Project. · View Herald TranscriptFeb 1 2019, 2:45 AM
Herald added a subscriber: guansong. · View Herald Transcript
smateo updated this revision to Diff 184709.Feb 1 2019, 2:47 AM
This revision is now accepted and ready to land.Feb 1 2019, 6:23 AM
smateo added a comment.Feb 1 2019, 6:24 AM

Thanks for the review, I will upload it shortly!

This revision was automatically updated to reflect the committed changes.