[libcxx-commits] [clang] [libcxx] [clang] [libc++] P3309 constexpr atomic and atomic ref [WIP] (PR #98738)

Nikolas Klauser via libcxx-commits libcxx-commits at lists.llvm.org
Sat Jul 13 05:00:23 PDT 2024


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


More information about the libcxx-commits mailing list