This is an archive of the discontinued LLVM Phabricator instance.

[libFuzzer] Do not use LLVM ostream in libFuzzer tests.
ClosedPublic

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

Details

Summary

LibFuzzer aims to have no LLVM dependencies, yet using gtest introduces a dependency on LLVM ostream, unless explicitly requested not to.
This change removes the dependency, and makes potential subsequent build system change much easier.

Diff Detail

Event Timeline

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

LGTM
libFuzzer *must* not have LLVM deps, but tests can.
But in this case there is clearly no reason to use custom LLVM ostream

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.