[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