This is an archive of the discontinued LLVM Phabricator instance.

[tsan] Fix and re-enable user_malloc.cc test
ClosedPublic

Authored by vitalybuka on May 1 2019, 11:30 AM.

Diff Detail

Repository
rL LLVM

Event Timeline

vitalybuka created this revision.May 1 2019, 11:30 AM
Herald added projects: Restricted Project, Restricted Project. · View Herald TranscriptMay 1 2019, 11:30 AM
Herald added subscribers: llvm-commits, Restricted Project, kubamracek. · View Herald Transcript
hctim accepted this revision.May 1 2019, 12:44 PM

LGTM with nits.

compiler-rt/test/tsan/Linux/user_malloc.cc
12 ↗(On Diff #197602)

Nit: is the l supposed to be here?

This revision is now accepted and ready to land.May 1 2019, 12:44 PM
vitalybuka updated this revision to Diff 197647.May 1 2019, 2:50 PM
vitalybuka marked an inline comment as done.

typo

This revision was automatically updated to reflect the committed changes.