This is an archive of the discontinued LLVM Phabricator instance.

Move #include manipulation code to new lib/Tooling/Inclusions.
ClosedPublic

Authored by ioeric on May 18 2018, 7:14 AM.

Diff Detail

Repository
rL LLVM

Event Timeline

ioeric created this revision.May 18 2018, 7:14 AM
ilya-biryukov accepted this revision.May 18 2018, 7:18 AM

LGTM with a minor naming suggestion.

include/clang/Format/Format.h
20 ↗(On Diff #147504)

Maybe Tooling/Includes?
I think inclusions is somewhat uncommon, while includes is more widely used. Don't have a strong opinion, though.

This revision is now accepted and ready to land.May 18 2018, 7:18 AM
ioeric marked an inline comment as done.May 18 2018, 7:19 AM
ioeric added inline comments.
include/clang/Format/Format.h
20 ↗(On Diff #147504)

I decided to use Inclusions over Includes or Headers because those two can be confused with directories that contain headers for the library.

This revision was automatically updated to reflect the committed changes.
ioeric marked an inline comment as done.
ioeric reopened this revision.May 18 2018, 11:39 AM

This was reverted in r332751 due to build bot failure with module config. I don't see how this failed as the directory setup is very similar to Tooling/Core/. Will investigate further next week.

This revision is now accepted and ready to land.May 18 2018, 11:39 AM
ioeric updated this revision to Diff 149091.May 30 2018, 4:54 AM
  • Fix module build.
This revision was automatically updated to reflect the committed changes.
cfe/trunk/lib/Tooling/CMakeLists.txt