Paul Fultz II

When LLVM is not found cmake says: ``` Add the installation prefix of "LLVM" to CMAKE_PREFIX_PATH or set "LLVM_DIR" to a directory containing one of the above files. ``` However,...

When compiling this code with hcc: ```cpp struct b { int c[0]; __attribute__((hc)) auto d(b) {} }; __attribute__((hc)) void e() { b a; a.d({{}}); } ``` It crashes with: ```...

Check for LLVM_EXTERNAL before calling find_package

1

Otherwise this wont find llvm when built independently.

This can help with integrated builds to disable installation of dependencies. When set to off, `bcm_install_targets` will be a no-op. This can probably even be a target-level property.

These properties are only named with `INTERFACE_` so they are whitelisted for interface libraries. Instead there should be a corresponding `bcm_target_` function to set the property.

Something like: ``` set_target_properties(${EXPORT_TARGETS} PROPERTIES DEBUG_POSTFIX "-d" SO_VERSION ${EXPORT_VERSION} VERSION ${EXPORT_VERSION} ) install(TARGETS ${EXPORT_TARGETS} EXPORT ${EXPORT_SET} ARCHIVE DESTINATION "${CMAKE_INSTALL_LIBDIR}" COMPONENT ${EXPORT_COMPONENT}_DEV LIBRARY NAMELINK_ONLY DESTINATION "${CMAKE_INSTALL_LIBDIR}" COMPONENT ${EXPORT_COMPONENT}_DEV LIBRARY CONFIGURATIONS Debug...

See this comment from @eldiener: > Why not have the line: > > add_library(${PARSE_NAME} STATIC EXCLUDE_FROM_ALL ${SOURCES}) > > be instead: > > add_library(${PARSE_NAME} OBJECT EXCLUDE_FROM_ALL ${SOURCES}) > > if...

This is needed by boost.spirit examples.