[libcxx-commits] [libcxx] [libc++][modules] Remove a dependency of __type_traits/invoke.h on __utility (PR #106795)

[Louis Dionne via libcxx-commits]

ldionne wrote:

> I'd rather do that for now and I'm fine with having somewhat arbitrary modules for some time. If this is actually fundamentally necessary we can look into removing the cycle later, but I'm not yet convinced it actually is. Basically, I'd rather sacrifice a bit of modulemap simplicity than code readability at this point, since we don't yet really know where the modulemap journey will get us.

That suggestion seems to work (functionally). One downside is that we won't catch the reintroduction of cycles between these submodules, allowing us to regress. But if we're careful not to add too many headers into this "core" module, perhaps it's OK.

I'll close this for now since we won't be going for this approach.

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