[libcxx-commits] [libcxx] [libc++] [NFC] Split typeinfo header (PR #68812)

Louis Dionne via libcxx-commits libcxx-commits at lists.llvm.org
Thu Oct 19 18:26:58 PDT 2023


ldionne wrote:

> > Sorry for confusing you, i'm not that familiar with how such things are handled.
> > Do i understand you correctly, that instead of splitting the header i should just remove the conditional `#include <exception>` from `typeinfo` and reverse the dependency (now `exception` would include `typeinfo`) instead?
> > If that's the case i would just do that in #65534 and close this PR.
> 
> Yes, that's at least what I would do. We don't have any exact policy on this I think, but this seems to me like a reasonable potentially breaking change. These headers aren't used that much compared to other headers anyways. Sorry that I didn't spot this earlier. @ldionne do you agree?

That sounds like a reasonable not-too-badly-breaking change.

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


More information about the libcxx-commits mailing list