[libcxx-commits] [PATCH] D135663: [libunwind] Install the headers by default

Louis Dionne via Phabricator via libcxx-commits libcxx-commits at lists.llvm.org
Wed May 10 09:38:00 PDT 2023


ldionne added a comment.

In D135663#4187545 <https://reviews.llvm.org/D135663#4187545>, @barannikov88 wrote:

> clang's unwind.h also gets installed and is found first when included from a source file (and it includes_next the libunwind's version, but only for apple platforms).
> Not sure I understand the logic, may it be somehow related to the fact that clang driver supports different unwind libraries?
> Either way, installing libunwind's header by default does not update clang's version, so the issue persists.

I think https://github.com/llvm/llvm-project/commit/032d422 is relevant here. It seems to be intentional that Clang doesn't use `unwind.h` outside of Apple.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D135663



More information about the libcxx-commits mailing list