This is an archive of the discontinued LLVM Phabricator instance.

Build clang-headers when building clang tools.
ClosedPublic

Authored by hokein on Oct 1 2018, 3:25 AM.

Details

Summary

clang tools require clang headers to work on real project, e.g. when we
build clangd via ninja clangd, we expect the binary can run on
real-world project (without running another command ninja clang-headers).

Diff Detail

Event Timeline

hokein created this revision.Oct 1 2018, 3:25 AM
sammccall accepted this revision.Oct 1 2018, 5:07 AM

> when we build clangd via ninja clangd, we expect the binary can run on real-world project

FWIW, I don't particularly have this expectation - I expect to be able to run the tests, and install the built binary.

But in practice this doesn't seem to slow things down much.

This revision is now accepted and ready to land.Oct 1 2018, 5:08 AM
This revision was automatically updated to reflect the committed changes.