This is an archive of the discontinued LLVM Phabricator instance.

Teach llvm_add_library() to search arbitrary directories for includes.
ClosedPublic

Authored by zturner on Feb 5 2015, 11:13 PM.

Details

Summary
Since header files are not compilation units, CMake does not require
you to specify them in the CMakeLists.txt file.  As a result, unless
a header file is explicitly added, CMake won't know about it, and
when generating IDE-based projects, CMake won't put the header files
into the IDE project.  LLVM currently tries to deal with this in two
ways:

  1) It looks for all .h files that are in the project directory,
     and adds those.
  2) llvm_add_library() understands the ADDITIONAL_HEADERS argument,
     which allows one to list an arbitrary list of headers.

This patch takes things one step further.  It adds the ability for
llvm_add_library() to take an ADDITIONAL_HEADER_DIRS argument,
which will specify a list of folders which CMake will glob for
header files.  Furthermore, it will glob not only for .h files, but
also for .inc files.

Included in this CL is an update to one of the existing users of
ADDITIONAL_HEADERS to use this new argument instead, to serve as an
illustration of how this cleans up the CMake.

The big advantage of this new approach is that until now, there was
no way for the IDE projects to locate the header files that are in
the include tree.  In other words, if you are in, for example,
lib/DebugInfo/DWARF, the corresponding includes for this project
will be located under include/DebugInfo/DWARF.  Now, in the
CMakeLists.txt for lib/DebugInfo/DWARF, you can simply write:

  ADDITIONAL_HEADER_DIRS
  ../../include/DebugInfo/DWARF

as an argument to llvm_add_library(), and all header files will get 
added to the IDE project.

Diff Detail

Event Timeline

zturner updated this revision to Diff 19463.Feb 5 2015, 11:13 PM
zturner retitled this revision from to Teach llvm_add_library() to search arbitrary directories for includes..
zturner updated this object.
zturner edited the test plan for this revision. (Show Details)
zturner added a reviewer: chandlerc.
zturner updated this object.
zturner added a subscriber: Unknown Object (MLST).

friendly ping to bring this to the top of your inbox

beanz accepted this revision.Feb 9 2015, 4:37 PM
beanz added a reviewer: beanz.
beanz added a subscriber: beanz.

LGTM. This seems like general goodness that reduces the complexity of our CMake files, and I verified it works as expected for Xcode.

Can you please cleanup any uses of ADDITIONAL_HEADERS that make sense?

This revision is now accepted and ready to land.Feb 9 2015, 4:37 PM

Yes, will probably do that as a followup, and simultaneously deprecate the
use of ADDITIONAL_HEADERS.

If someone really wanted to get clever, they could also update
llvm_add_library to translate the source directory to a corresponding
include directory and automatically append it to ADDITIONAL_HEADER_DIRS.
This would make all IDE projects way better in my opinion.

This revision was automatically updated to reflect the committed changes.