[all-commits] [llvm/llvm-project] 45964e: [libc++] [test] Fix `__has_include` usage, expand ...

[Stephan T. Lavavej via All-commits]

  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: 45964eb9b88c46045e4e84beb4e2135cdeed6855
      https://github.com/llvm/llvm-project/commit/45964eb9b88c46045e4e84beb4e2135cdeed6855
  Author: Stephan T. Lavavej <stl at nuwen.net>
  Date:   2024-06-02 (Sun, 02 Jun 2024)

  Changed paths:
    M libcxx/test/std/concepts/concepts.compare/concepts.totallyordered/totally_ordered.pass.cpp
    M libcxx/test/std/concepts/concepts.compare/concepts.totallyordered/totally_ordered_with.pass.cpp
    M libcxx/test/std/thread/thread.condition/thread.condition.condvarany/wait_for_pred.pass.cpp
    M libcxx/test/support/test_macros.h

  Log Message:
  -----------
  [libc++] [test] Fix `__has_include` usage, expand condvarany and spaceship coverage (#94120)

Three unrelated, small improvements:

* `test_macros.h` was incorrectly saying `__has_include("<version>")`
instead of `__has_include(<version>)`.
+ This caused `<ciso646>` to always be included (noticed because MSVC's
STL emitted a deprecation warning).
  + I searched all of LLVM and found no other occurrences.
* `thread.condition.condvarany/wait_for_pred.pass.cpp` forgot to test
anything.
  + I followed what `wait_for.pass.cpp` is testing.
* Uncomment spaceship test coverage.



To unsubscribe from these emails, change your notification settings at https://github.com/llvm/llvm-project/settings/notifications