[libcxx-commits] [libcxxabi] [libcxxabi] Update ItaniumDemangle.h from LLVM (PR #140273)

[Aiden Grossman via libcxx-commits]

boomanaiden154 wrote:

> Sorry, the problem is that from my pov ItaniumMangle was in llvm, so I'm trying to understand what the rules for limiting its inclusion of llvm headers.

It's the same across libcxxabi and LLVM (or at least should be), so cannot depend on stuff within LLVM.

> By lib headers, I meant ItaniumMangle is in the llvm/include/llvm directory, and it sounds like that's for api/library headers, so including things like llvm/Support isn't safe/allowed as those are internal/implementation headers.
>
> Is that correct?

You should be fine including things in `llvm/include/llvm` with the obvious exceptions like you cannot create circular dependencies. Demangling is a special case given it's shared across the monorepo.

https://github.com/llvm/llvm-project/pull/140273