Skip to content
This repository has been archived by the owner on Nov 1, 2023. It is now read-only.

Set TSan options based on external symbolizer #1787

Merged
merged 3 commits into from
Apr 25, 2022
Merged

Set TSan options based on external symbolizer #1787

merged 3 commits into from
Apr 25, 2022

Conversation

ranweiler
Copy link
Member

@ranweiler ranweiler commented Apr 12, 2022 •
edited

  • Add LlvmSymbolizer wrapper around validated paths to external llvm-symbolizer exes
  • Pivot sanitizer environment variable generation on LlvmSymbolizer
  • Set external symbolizer path in TSAN_OPTIONS

Closes #1770.

@ranweiler ranweiler changed the title Set TSAN options based on external symbolizer Set TSan options based on external symbolizer Apr 12, 2022
@ranweiler ranweiler mentioned this pull request Apr 13, 2022
Open
@ranweiler ranweiler merged commit 7fe9e37 into microsoft:main Apr 25, 2022
@ranweiler ranweiler deleted the tsan-options branch April 25, 2022 20:00
@ghost ghost locked as resolved and limited conversation to collaborators May 26, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@chkeita chkeita chkeita approved these changes

@stishkin stishkin stishkin approved these changes

@hayleycall hayleycall hayleycall approved these changes

Assignees
No one assigned
Labels
passed-integration-tests
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Add first-tier support for ThreadSanitizer
4 participants
@ranweiler @chkeita @stishkin @hayleycall