This is an archive of the discontinued LLVM Phabricator instance.

[libcxx] Use mtx_plain | mtx_recursive following C11 API
ClosedPublic

Authored by phosek on Jan 15 2020, 2:00 PM.

Details

Summary

The C11 API specifies that to initialize a recursive mutex,
mtx_plain | mtx_recursive should be used with mtx_init.

Diff Detail

Event Timeline

phosek created this revision.Jan 15 2020, 2:00 PM
Herald added a project: Restricted Project. · View Herald TranscriptJan 15 2020, 2:00 PM
EricWF accepted this revision.Jan 15 2020, 3:13 PM
EricWF added a subscriber: hans.

LGTM.

We should backport this into the LLVM 10 release.
@hans Is that OK by you?

This revision is now accepted and ready to land.Jan 15 2020, 3:13 PM
This revision was automatically updated to reflect the committed changes.
hans added a comment.Jan 17 2020, 1:08 AM

LGTM.

We should backport this into the LLVM 10 release.
@hans Is that OK by you?

Yes. Cherry-picked in 729258062afeb587fcfaeebb76d5a0ff69cf58b9