This is an archive of the discontinued LLVM Phabricator instance.

include compiler-rt in build-llvm.pl
ClosedPublic

Authored by kubamracek on Oct 10 2014, 5:33 PM.

Details

Reviewers
kubamracek
Summary

The new test cases for ASan fail if the llvm build that is used with LLDB doesn't have compiler-rt (because the resulting compiler then cannot build with -fsanitize=address). Let's include compiler-rt in build-llvm.pl script and make sure we actually *build* it by removing the NO_RUNTIME_LIBS=1 argument used in the make line. After this, the ASan tests pass on a fresh svn checkout.

Diff Detail

Event Timeline

kubamracek updated this revision to Diff 14759.Oct 10 2014, 5:33 PM
kubamracek retitled this revision from to include compiler-rt in build-llvm.pl.
kubamracek updated this object.
kubamracek edited the test plan for this revision. (Show Details)
kubamracek added subscribers: Unknown Object (MLST), jasonmolenda, jingham.

Cool, please commit.

kubamracek accepted this revision.Oct 10 2014, 6:35 PM
kubamracek added a reviewer: kubamracek.

Landed in r219555!

This revision is now accepted and ready to land.Oct 10 2014, 6:35 PM
kubamracek closed this revision.Oct 13 2014, 9:26 AM