[Lldb-commits] [PATCH] D31969: [CMake] Support generating Config.h
Zachary Turner via Phabricator via lldb-commits
lldb-commits at lists.llvm.org
Tue Apr 11 19:01:37 PDT 2017
zturner added a comment.
How much would it complicate things to move the hand maintained files out of tree? If the Xcode build isn't really a thing we're officially supporting, perhaps we can take that aggressive approach in-tree as well?
================
Comment at: cmake/modules/LLDBConfig.cmake:287-293
+ install(DIRECTORY ${CMAKE_CURRENT_BINARY_DIR}/include/
+ COMPONENT lldb_headers
+ DESTINATION include
+ FILES_MATCHING
+ PATTERN "*.h"
+ PATTERN ".svn" EXCLUDE
+ PATTERN ".cmake" EXCLUDE
----------------
Is this necessary?
================
Comment at: include/lldb/Host/Config.h.cmake:10-11
+
+#ifndef liblldb_Config_h_
+#define liblldb_Config_h_
+
----------------
Since we're in here anyway, can you use a fully qualified include guard here? Like `#ifndef LLDB_HOST_CONFIG_H`?
https://reviews.llvm.org/D31969
More information about the lldb-commits
mailing list