[all-commits] [llvm/llvm-project] 164c20: [libc++][test] Fix simple warnings (#74186)

Stephan T. Lavavej via All-commits all-commits at lists.llvm.org
Tue Dec 5 06:46:55 PST 2023


  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: 164c204a19f7859d570003d4c5e82faf48cb65a9
      https://github.com/llvm/llvm-project/commit/164c204a19f7859d570003d4c5e82faf48cb65a9
  Author: Stephan T. Lavavej <stl at nuwen.net>
  Date:   2023-12-05 (Tue, 05 Dec 2023)

  Changed paths:
    M libcxx/test/std/algorithms/alg.modifying.operations/alg.move/move.pass.cpp
    M libcxx/test/std/algorithms/alg.modifying.operations/alg.move/move_backward.pass.cpp
    M libcxx/test/std/algorithms/alg.nonmodifying/alg.count/count.pass.cpp
    M libcxx/test/std/containers/views/mdspan/ConvertibleToIntegral.h
    M libcxx/test/std/numerics/rand/rand.device/ctor.pass.cpp
    M libcxx/test/std/strings/string.view/string.view.comparison/equal.pass.cpp
    M libcxx/test/std/strings/string.view/string.view.comparison/greater.pass.cpp
    M libcxx/test/std/strings/string.view/string.view.comparison/greater_equal.pass.cpp
    M libcxx/test/std/strings/string.view/string.view.comparison/less.pass.cpp
    M libcxx/test/std/strings/string.view/string.view.comparison/less_equal.pass.cpp
    M libcxx/test/std/strings/string.view/string.view.comparison/not_equal.pass.cpp
    M libcxx/test/std/thread/thread.mutex/thread.lock/thread.lock.scoped/mutex.pass.cpp
    M libcxx/test/std/thread/thread.mutex/thread.mutex.requirements/thread.shared_mutex.requirements/thread.shared_mutex.class/default.pass.cpp
    M libcxx/test/support/test_macros.h

  Log Message:
  -----------
  [libc++][test] Fix simple warnings (#74186)

Found while running libc++'s tests with MSVC's STL. This fixes 3 kinds of warnings:

- Add void-casts to fix `-Wunused-variable` warnings.
- Avoid sign/truncation warnings in `ConvertibleToIntegral.h`.
- Add `TEST_STD_AT_LEAST_23_OR_RUNTIME_EVALUATED` to avoid mixing preprocessor 
  and runtime tests.
- Cleanup: Add `TEST_STD_AT_LEAST_20_OR_RUNTIME_EVALUATED` for
  consistency.




More information about the All-commits mailing list