This is an archive of the discontinued LLVM Phabricator instance.

[libFuzzer] Add a dependency on symbolizer from libFuzzer tests
ClosedPublic

Authored by george.karpenkov on Jul 12 2017, 10:51 AM.

Details

Summary

Some libFuzzer tests on Linux would fail with bizarre error messages unless llvm-symbolizer binary is present.
I strongly suspect that CI passes because the machine already has llvm-symbolizer installed in $PATH.

Diff Detail

Repository
rL LLVM

Event Timeline

kcc accepted this revision.Jul 12 2017, 10:51 AM

LGTM, thanks!

This revision is now accepted and ready to land.Jul 12 2017, 10:51 AM
This revision was automatically updated to reflect the committed changes.
MatzeB added a subscriber: MatzeB.Jul 17 2017, 11:04 AM

I think this broke the greendragon libfuzzer build: http://lab.llvm.org:8080/green/job/libFuzzer/5100/ this one seems like the most likely cause.
Any idea how to get this back to green?