This is an archive of the discontinued LLVM Phabricator instance.

[LLVM] Moving libFuzzer from LLVM to compiler-rt
ClosedPublic

Authored by george.karpenkov on Aug 21 2017, 1:37 PM.

Details

Summary

This PR is tightly coupled to https://reviews.llvm.org/D36908 and https://reviews.llvm.org/D36909.
Per Kostya's request removes CMake and tests, and temporarily leaves the sources and README.txt in the old location.

Diff Detail

Repository
rL LLVM

Event Timeline

kcc edited edge metadata.Aug 21 2017, 1:39 PM

LGTM

vitalybuka added inline comments.Aug 21 2017, 1:40 PM
lib/CMakeLists.txt
23 ↗(On Diff #112053)

Probably you should revert this one

@vitalybuka it wouldn't work without the CMake machinery. If I re-add CMake machinery, it would break the new code in compiler-rt, as we would have colliding target names.

vitalybuka accepted this revision.Aug 21 2017, 2:10 PM
This revision is now accepted and ready to land.Aug 21 2017, 2:10 PM

Shouldn't the title be the other way around?

george.karpenkov retitled this revision from [LLVM] Moving libFuzzer from compiler-rt to LLVM to [LLVM] Moving libFuzzer from LLVM to compiler-rt.Aug 21 2017, 2:16 PM
This revision was automatically updated to reflect the committed changes.
llvm/trunk/lib/Fuzzer/test/value-profile-load.test