This is an archive of the discontinued LLVM Phabricator instance.

[sanitizers] lsan+sancov doesn't need ubsanrt (multi def error)
ClosedPublic

Authored by aizatsky on Dec 8 2016, 2:22 PM.

Event Timeline

aizatsky updated this revision to Diff 80825.Dec 8 2016, 2:22 PM
aizatsky retitled this revision from to [sanitizers] lsan+sancov doesn't need ubsanrt (multi def error).
aizatsky updated this object.
aizatsky added a reviewer: eugenis.
eugenis accepted this revision.Dec 8 2016, 2:29 PM
eugenis edited edge metadata.

LGTM

test/Driver/sanitizer-ld.c
304

please add another "-NOT: libclang_rt.ubsan" line after libclang_rt.lsan

This revision is now accepted and ready to land.Dec 8 2016, 2:29 PM
aizatsky updated this revision to Diff 80828.Dec 8 2016, 2:30 PM
aizatsky marked an inline comment as done.
aizatsky edited edge metadata.

one more CHECK

This revision was automatically updated to reflect the committed changes.