Added a separate install target for compilert-rt-headers

[ilya-biryukov]

Summary

This patch adds a new install target install-compilert-rt-headers,
that is similar to install-clang-headers.

It allows to install the headers without installing all of
compiler-rt.

[ilya-biryukov]

This is a resubmission of D41164, which was accidentally "commanded" back from me.

[beanz]

This patch LGTM.

[smeenai]

Isn't this gonna break building compiler-rt standalone (i.e. without LLVM)? Or does the include/ directory already rely on LLVM in some other way? See the history of D40687.

[ilya-biryukov]

In D41244#955511, @smeenai wrote:

Isn't this gonna break building compiler-rt standalone (i.e. without LLVM)? Or does the include/ directory already rely on LLVM in some other way? See the history of D40687.

I was not aware of this use-case, thanks for mentioning it. It would definitely break without LLVM.
I'll update the patch to not use add_llvm_install_targets.

[ilya-biryukov]

• Use add_custom_target directly instead of add_llvm_install_targets

[ilya-biryukov]

Removed a usage of add_llvm_install_targets, so it should now work for standalone builds too.

[ilya-biryukov]

I double-checked that the new targets work with standalone build before submitting.

include/CMakeLists.txt

 if (COMPILER_RT_BUILD_XRAY)
   set(XRAY_HEADERS
     xray/xray_interface.h
     xray/xray_log_interface.h)
 endif(COMPILER_RT_BUILD_XRAY)
 
 set(COMPILER_RT_HEADERS
   ${SANITIZER_HEADERS}
-  ${XRAY_HEADERS})
+  ${XRAY_HEADERS})
 
 set(output_dir ${COMPILER_RT_OUTPUT_DIR}/include)
 
 # Copy compiler-rt headers to the build tree.
 set(out_files)
 foreach( f ${COMPILER_RT_HEADERS} )
   set( src ${CMAKE_CURRENT_SOURCE_DIR}/${f} )
   set( dst ${output_dir}/${f} )
   add_custom_command(OUTPUT ${dst}
     DEPENDS ${src}
     COMMAND ${CMAKE_COMMAND} -E copy_if_different ${src} ${dst}
     COMMENT "Copying compiler-rt's ${f}...")
   list(APPEND out_files ${dst})
 endforeach( f )
 
 add_custom_target(compiler-rt-headers ALL DEPENDS ${out_files})
 add_dependencies(compiler-rt compiler-rt-headers)
 set_target_properties(compiler-rt-headers PROPERTIES FOLDER "Compiler-RT Misc")
 
 # Install sanitizer headers.
 install(FILES ${SANITIZER_HEADERS}
+  COMPONENT compiler-rt-headers
   PERMISSIONS OWNER_READ OWNER_WRITE GROUP_READ WORLD_READ
   DESTINATION ${COMPILER_RT_INSTALL_PATH}/include/sanitizer)
 # Install xray headers.
 install(FILES ${XRAY_HEADERS}
+  COMPONENT compiler-rt-headers
   PERMISSIONS OWNER_READ OWNER_WRITE GROUP_READ WORLD_READ
   DESTINATION ${COMPILER_RT_INSTALL_PATH}/include/xray)
+
+if (NOT CMAKE_CONFIGURATION_TYPES) # don't add this for IDEs.
+  add_llvm_install_targets(install-compiler-rt-headers
+                           DEPENDS compiler-rt-headers
+                           COMPONENT compiler-rt-headers)
+endif()