This fixes the test on OS X by inheriting the default TSAN_OPTIONS (set in lit.cfg). Namely, we get abort_on_error=0, so the process doesn't crash.
Diff Detail
Diff Detail
Event Timeline
Comment Actions
LGTM because it can be useful to run tests with custom TSAN_OPTIONS supplied in env.
However, I wonder why do we enable abort_on_error on Mac default (it is not enabled on other OSes), and then disable it as more useful behavior. Won't users also suffer from abort_on_error=1?
I've hit it with Go runtime on Mac. It also lead to a bad behavior and I disabled it for Go on Mac.
Should not we just switch the default to 0?