[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