qiongsiwu1 added inline comments.
================ Comment at: clang/lib/Headers/CMakeLists.txt:421 install( FILES ${ppc_wrapper_files} ---------------- craig.topper wrote: > There appear to be two installs of ppc_wrapper_files with different > components. Is that intentional? Ah yes this is indeed intentional. `cuda_wrapper_files` and `openmp_wrapper_files` have two install targets as well. The first target is part of the "catch-all" `clang-resource-headers`. The second targets are not installed by default (`EXCLUDE_FROM_ALL`). and a distribution build can opt-in the relevant set of headers. That said, if there are better ways to implement the logic I am all ears. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D123498/new/ https://reviews.llvm.org/D123498 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits