qiongsiwu1 created this revision.
qiongsiwu1 added reviewers: craig.topper, DavidSpickett, nemanjai.
qiongsiwu1 added a project: clang.
Herald added subscribers: StephenFan, guansong, kristof.beyls, yaxunl, mgorny.
Herald added a project: All.
qiongsiwu1 requested review of this revision.
Herald added a reviewer: jdoerfert.
Herald added subscribers: cfe-commits, sstefan1.
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.
Repository:
rG LLVM Github Monorepo
https://reviews.llvm.org/D126002
Files:
clang/lib/Headers/CMakeLists.txt
Index: clang/lib/Headers/CMakeLists.txt
===================================================================
--- clang/lib/Headers/CMakeLists.txt
+++ clang/lib/Headers/CMakeLists.txt
@@ -366,7 +366,7 @@
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"
@@ -387,7 +387,7 @@
# 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}")
@@ -546,7 +546,7 @@
install(
FILES ${openmp_wrapper_files}
- DESTINATION ${header_install_dir}
+ DESTINATION ${header_install_dir}/openmp_wrappers
EXCLUDE_FROM_ALL
COMPONENT openmp-resource-headers)
@@ -557,7 +557,7 @@
COMPONENT utility-resource-headers)
install(
- FILES ${windows_files}
+ FILES ${windows_only_files}
DESTINATION ${header_install_dir}
EXCLUDE_FROM_ALL
COMPONENT windows-resource-headers)
@@ -621,7 +621,7 @@
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
Index: clang/lib/Headers/CMakeLists.txt
===================================================================
--- clang/lib/Headers/CMakeLists.txt
+++ clang/lib/Headers/CMakeLists.txt
@@ -366,7 +366,7 @@
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"
@@ -387,7 +387,7 @@
# 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}")
@@ -546,7 +546,7 @@
install(
FILES ${openmp_wrapper_files}
- DESTINATION ${header_install_dir}
+ DESTINATION ${header_install_dir}/openmp_wrappers
EXCLUDE_FROM_ALL
COMPONENT openmp-resource-headers)
@@ -557,7 +557,7 @@
COMPONENT utility-resource-headers)
install(
- FILES ${windows_files}
+ FILES ${windows_only_files}
DESTINATION ${header_install_dir}
EXCLUDE_FROM_ALL
COMPONENT windows-resource-headers)
@@ -621,7 +621,7 @@
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
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
