This is an archive of the discontinued LLVM Phabricator instance.

[lsan] Do not check for leaks in the forked process
AbandonedPublic

Authored by vitalybuka on Jun 4 2018, 6:13 PM.

Diff Detail

Event Timeline

vitalybuka updated this revision to Diff 149886.Jun 4 2018, 6:13 PM
vitalybuka created this revision.

typo

vitalybuka updated this revision to Diff 149887.Jun 4 2018, 6:16 PM

test cleanup

alekseyshl accepted this revision.Jun 5 2018, 10:26 AM
alekseyshl added inline comments.
compiler-rt/lib/lsan/lsan_common.cc
577

It might be a good idea to print something to the log explaining that lsan is pretty much disabled. On the other hand, not sure how spammy that would be. Up to you.

This revision is now accepted and ready to land.Jun 5 2018, 10:26 AM
This revision was automatically updated to reflect the committed changes.
vitalybuka marked an inline comment as done.
orivej added a subscriber: orivej.Aug 24 2018, 10:35 PM
vitalybuka reopened this revision.Aug 27 2018, 10:29 AM
This revision is now accepted and ready to land.Aug 27 2018, 10:29 AM
vitalybuka planned changes to this revision.Aug 27 2018, 10:29 AM
This revision was not accepted when it landed; it landed in state Changes Planned.Oct 7 2019, 4:16 AM
This revision was automatically updated to reflect the committed changes.
Herald added projects: Restricted Project, Restricted Project. · View Herald TranscriptOct 7 2019, 4:16 AM
Herald added a subscriber: Restricted Project. · View Herald Transcript
vitalybuka reopened this revision.Oct 7 2019, 11:19 AM
This revision is now accepted and ready to land.Oct 7 2019, 11:19 AM
vitalybuka abandoned this revision.Oct 7 2019, 11:20 AM

It was reverted. But I am not planing to work on this soon.