philnik777 wrote:

> @philnik777 I would rather not, at least for the review, manipulating two 
> branches will be cumbersome. I can split it before merging, but as I don't 
> expect this to happen soon, it should be fine.

This is less about having two separate commits than having mostly unrelated 
changes in separate PRs. It's "the llvm project", but it's really not a single 
project. The people reviewing the clang part don't know what to look for in 
libc++ and vice versa. You can use tools like 
https://github.com/spacedentist/spr to make it easier to manage a stacked PR if 
you want.

https://github.com/llvm/llvm-project/pull/98738
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to