This is an archive of the discontinued LLVM Phabricator instance.

[BOLT] Change mutex implementation
ClosedPublic

Authored by Elvina on Jul 5 2022, 2:35 PM.

Details

Summary

Changed acquire implementation to __atomic_test_and_set() and release to __atomic_clear() so it eliminates inline asm usage and is arch independent.

Elvina Yakubova,
Advanced Software Technology Lab, Huawei

Diff Detail

Event Timeline

Elvina created this revision.Jul 5 2022, 2:35 PM
Herald added a reviewer: Amir. · View Herald Transcript
Herald added a project: Restricted Project. · View Herald Transcript
Herald added a subscriber: ayermolo. · View Herald Transcript
Elvina requested review of this revision.Jul 5 2022, 2:35 PM
Elvina edited the summary of this revision. (Show Details)
Elvina updated this revision to Diff 442401.Jul 5 2022, 2:42 PM
yota9 accepted this revision.Jul 5 2022, 2:54 PM

LGTM
@maksfb Please take a look :)

This revision is now accepted and ready to land.Jul 5 2022, 2:54 PM
rafauler accepted this revision.Jul 5 2022, 6:30 PM

LGTM

maksfb accepted this revision.Jul 5 2022, 8:51 PM
This revision was landed with ongoing or failed builds.Jul 5 2022, 10:24 PM
This revision was automatically updated to reflect the committed changes.