[clang] [clang] Fix missing installed header (PR #95979)

etcwilde wrote: For folks landing here, this should be fixed in CMake 4.1 by policy [CMP0187](https://gitlab.kitware.com/cmake/cmake/-/merge_requests/10271). https://github.com/llvm/llvm-project/pull/95979 ___ cfe-commits mailing list cfe-commits@list

[clang] [clang] Fix missing installed header (PR #95979)

etcwilde wrote: @danielotero Thank you for your contribution. https://github.com/llvm/llvm-project/pull/95979 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [clang] Fix missing installed header (PR #95979)

github-actions[bot] wrote: @danielotero Congratulations on having your first Pull Request (PR) merged into the LLVM Project! Your changes will be combined with recent changes from other authors, then tested by our [build bots](https://lab.llvm.org/buildbot/). If there is a problem with a bu

[clang] [clang] Fix missing installed header (PR #95979)

https://github.com/etcwilde closed https://github.com/llvm/llvm-project/pull/95979 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [clang] Fix missing installed header (PR #95979)

https://github.com/compnerd approved this pull request. https://github.com/llvm/llvm-project/pull/95979 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [clang] Fix missing installed header (PR #95979)

danielotero wrote: I forgot to say I don't have commit access, so feel free to merge this. https://github.com/llvm/llvm-project/pull/95979 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commi

[clang] [clang] Fix missing installed header (PR #95979)

mathstuf wrote: > Nice find. That's kind of horrifying. Indeed. CMake's old "guess the extension" behavior was always a…fun thing to deal with. Alas, this emergent behavior was unknown at the time and a straight fix is likely to break other projects :( . Plus it's not likely we'll get patched

[clang] [clang] Fix missing installed header (PR #95979)

https://github.com/etcwilde approved this pull request. Nice find. That's kind of horrifying. I'm fine with flipping the file order to avoid the source file caching bug in CMake. CC @compnerd since he also knows CMake and may have opinions. https://github.com/llvm/llvm-project/pull/95979 __

[clang] [clang] Fix missing installed header (PR #95979)

llvmbot wrote: @llvm/pr-subscribers-backend-x86 Author: Daniel Otero (danielotero) Changes Since commit 8d468c132eed7ffe34d601b224220efd51655eb3, the header `openmp_wrappers/complex` is hidden behind `openmp_wrappers/complex.h` due to a bug in CMake[^1], so is not actually installed. To

[clang] [clang] Fix missing installed header (PR #95979)

github-actions[bot] wrote: Thank you for submitting a Pull Request (PR) to the LLVM Project! This PR will be automatically labeled and the relevant teams will be notified. If you wish to, you can add reviewers by using the "Reviewers" section on this page. If this is not working for you, it

[clang] [clang] Fix missing installed header (PR #95979)

https://github.com/danielotero created https://github.com/llvm/llvm-project/pull/95979 Since commit 8d468c132eed7ffe34d601b224220efd51655eb3, the header `openmp_wrappers/complex` is hidden behind `openmp_wrappers/complex.h` due to a bug in CMake[^1], so is not actually installed. To test the