[PATCH] D123498: [clang] Adding Platform/Architecture Specific Resource Header Installation Targets

[Qiongsi Wu via Phabricator via cfe-commits]

qiongsiwu1 added inline comments.


================
Comment at: clang/lib/Headers/CMakeLists.txt:537
+  FILES ${openmp_wrapper_files}
+  DESTINATION ${header_install_dir}
+  EXCLUDE_FROM_ALL
----------------
craig.topper wrote:
> Should this be ${header_install_dir}/openmp_wrappers
Oh good catch! Thanks! I am fixing this in https://reviews.llvm.org/D126002, together with the issue with windows only headers. 


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D123498/new/

https://reviews.llvm.org/D123498