This is an archive of the discontinued LLVM Phabricator instance.

[tsan] Always initialize tsan when building shared lib
ClosedPublic

Authored by zhuhan0 on Jan 18 2023, 10:13 AM.

Details

Summary

https://reviews.llvm.org/D135925 fixed tsan to always initialize on Android, because there tsan is used as shared library.

We can generalize the fix to always initialize when building tsan shared library. This fixes the issue where we use tsan as a shared library on Linux and it failed to initialize.

Diff Detail

Event Timeline

zhuhan0 created this revision.Jan 18 2023, 10:13 AM
Herald added a project: Restricted Project. · View Herald TranscriptJan 18 2023, 10:13 AM
zhuhan0 requested review of this revision.Jan 18 2023, 10:13 AM
Herald added a project: Restricted Project. · View Herald TranscriptJan 18 2023, 10:13 AM
Herald added a subscriber: Restricted Project. · View Herald Transcript
zhuhan0 edited the summary of this revision. (Show Details)Jan 18 2023, 10:18 AM
zhuhan0 edited the summary of this revision. (Show Details)Jan 19 2023, 10:52 AM
dvyukov accepted this revision.Jan 19 2023, 11:37 PM
This revision is now accepted and ready to land.Jan 19 2023, 11:37 PM
This revision was automatically updated to reflect the committed changes.

Thanks for the review!