This is an archive of the discontinued LLVM Phabricator instance.

tsan: add AccessVptr
ClosedPublic

Authored by dvyukov on Aug 2 2021, 7:59 AM.

Details

Summary

Add AccessVptr access type.
For now it's converted to the same thr->is_vptr_access,
but later it will be passed directly to ReportRace
and will enable efficient tail calling in MemoryAccess function
(currently tsan_vptr_update/tsan_vptr_read can't use
tail calls in MemoryAccess because of the trailing assignment
to thr->is_vptr_access).

Depends on D107276.

Diff Detail

Event Timeline

dvyukov requested review of this revision.Aug 2 2021, 7:59 AM
dvyukov created this revision.
Herald added a project: Restricted Project. · View Herald TranscriptAug 2 2021, 7:59 AM
Herald added a subscriber: Restricted Project. · View Herald Transcript
vitalybuka accepted this revision.Aug 2 2021, 3:57 PM
This revision is now accepted and ready to land.Aug 2 2021, 3:57 PM
melver accepted this revision.Aug 3 2021, 1:03 AM
This revision was landed with ongoing or failed builds.Aug 3 2021, 2:03 AM
Closed by commit rG18c6ed2f0f29: tsan: add AccessVptr (authored by dvyukov). · Explain Why
This revision was automatically updated to reflect the committed changes.