[libcxx-commits] [libcxx] [libc++] Refactor memory allocation in basic_string (PR #128423)

[Louis Dionne via libcxx-commits]

================
@@ -122,7 +113,13 @@ TEST_CONSTEXPR_CXX20 bool test() {
     assert(s2 == p2);
 
     imp2.deactivate();
-    test_assign(s1, s2);
+
----------------
ldionne wrote:

It seems to me this should be a separate NFC patch. Otherwise it looks like this was a necessary change to make your patch pass CI, and that would be surprising.

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