Skip to content

Commit

Permalink
[clang] Fixing arm-common, windows only and openmp header install tar…
Browse files Browse the repository at this point in the history
…gets

https://reviews.llvm.org/D123498 contains a few errors resulting in incorrect target contents or mismatched target/list names. This patch fixes all the known errors.

Reviewed By: craig.topper

Differential Revision: https://reviews.llvm.org/D126002
  • Loading branch information
qiongsiwu committed May 20, 2022
1 parent 9af56c6 commit 1f12718
Showing 1 changed file with 5 additions and 5 deletions.
10 changes: 5 additions & 5 deletions clang/lib/Headers/CMakeLists.txt
Expand Up @@ -366,7 +366,7 @@ set_target_properties("clang-resource-headers" PROPERTIES
RUNTIME_OUTPUT_DIRECTORY "${output_dir}")
add_dependencies("clang-resource-headers"
"core-resource-headers"
"arm-common-headers"
"arm-common-resource-headers"
"arm-resource-headers"
"aarch64-resource-headers"
"cuda-resource-headers"
Expand All @@ -387,7 +387,7 @@ add_dependencies("clang-resource-headers"

# Core/common headers
add_header_target("core-resource-headers" ${core_files})
add_header_target("arm-common-headers" "${arm_common_files};${arm_common_generated_files}")
add_header_target("arm-common-resource-headers" "${arm_common_files};${arm_common_generated_files}")

# Architecture/platform specific targets
add_header_target("arm-resource-headers" "${arm_only_files};${arm_only_generated_files}")
Expand Down Expand Up @@ -546,7 +546,7 @@ install(

install(
FILES ${openmp_wrapper_files}
DESTINATION ${header_install_dir}
DESTINATION ${header_install_dir}/openmp_wrappers
EXCLUDE_FROM_ALL
COMPONENT openmp-resource-headers)

Expand All @@ -557,7 +557,7 @@ install(
COMPONENT utility-resource-headers)

install(
FILES ${windows_files}
FILES ${windows_only_files}
DESTINATION ${header_install_dir}
EXCLUDE_FROM_ALL
COMPONENT windows-resource-headers)
Expand Down Expand Up @@ -621,7 +621,7 @@ if (NOT LLVM_ENABLE_IDE)
DEPENDS openmp-resource-headers
COMPONENT openmp-resource-headers)
add_llvm_install_targets(install-windows-resource-headers
DEPENDS window-resource-headers
DEPENDS windows-resource-headers
COMPONENT windows-resource-headers)
add_llvm_install_targets(install-utility-resource-headers
DEPENDS utility-resource-headers
Expand Down

0 comments on commit 1f12718

Please sign in to comment.