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

via libcxx-commits libcxx-commits at lists.llvm.org
Tue Oct 17 07:13:08 PDT 2023


philnik777 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?

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


More information about the libcxx-commits mailing list