[all-commits] [llvm/llvm-project] 9dc854: [libc++] Improve test coverage for copy/move ctors...
Peng Liu via All-commits
all-commits at lists.llvm.org
Thu Mar 20 05:38:01 PDT 2025
Branch: refs/heads/main
Home: https://github.com/llvm/llvm-project
Commit: 9dc854ccbd3d29c8dcf6b81b691724c9fc4bfc6c
https://github.com/llvm/llvm-project/commit/9dc854ccbd3d29c8dcf6b81b691724c9fc4bfc6c
Author: Peng Liu <winner245 at hotmail.com>
Date: 2025-03-20 (Thu, 20 Mar 2025)
Changed paths:
M libcxx/test/std/containers/sequences/vector.bool/copy.pass.cpp
M libcxx/test/std/containers/sequences/vector.bool/copy_alloc.pass.cpp
M libcxx/test/std/containers/sequences/vector.bool/move.pass.cpp
M libcxx/test/std/containers/sequences/vector.bool/move_alloc.pass.cpp
Log Message:
-----------
[libc++] Improve test coverage for copy/move ctors for vector<bool> (#120132)
The current tests for `vector<bool>` fail to adequately cover realistic
scenarios, as they are limited to cases of up to 3 bytes, representing
less than 1/2 of a word size on a 64-bit system. However, most
`vector<bool>` operations rely on code paths triggered only when
handling multiple storage words (8 bytes each). To address this gap,
this PR rewrites the tests for copy and move constructors, as well as
allocator-extended copy and move constructors, ensuring that previously
untested code paths are now thoroughly validated.
To unsubscribe from these emails, change your notification settings at https://github.com/llvm/llvm-project/settings/notifications
More information about the All-commits
mailing list