[all-commits] [llvm/llvm-project] aec82f: [libc++] Require the use of clang-verify in .fail....

[Louis Dionne via All-commits]

  Branch: refs/heads/master
  Home:   https://github.com/llvm/llvm-project
  Commit: aec82f9256a559f5884420920c1956e60a1474d9
      https://github.com/llvm/llvm-project/commit/aec82f9256a559f5884420920c1956e60a1474d9
  Author: Louis Dionne <ldionne at apple.com>
  Date:   2020-03-25 (Wed, 25 Mar 2020)

  Changed paths:
    M libcxx/test/libcxx/depr/depr.default.allocator/allocator.members/allocate.cxx2a.fail.cpp
    M libcxx/test/libcxx/diagnostics/enable_nodiscard.fail.cpp
    M libcxx/test/libcxx/diagnostics/enable_nodiscard_disable_after_cxx17.fail.cpp
    M libcxx/test/libcxx/diagnostics/enable_nodiscard_disable_nodiscard_ext.fail.cpp
    M libcxx/test/libcxx/diagnostics/nodiscard_aftercxx17.fail.cpp
    M libcxx/test/libcxx/diagnostics/nodiscard_extensions.fail.cpp
    M libcxx/test/libcxx/thread/thread.lock/thread.lock.guard/nodiscard.fail.cpp
    M libcxx/test/std/containers/associative/map/map.access/empty.fail.cpp
    M libcxx/test/std/containers/associative/multimap/empty.fail.cpp
    M libcxx/test/std/containers/associative/multiset/empty.fail.cpp
    M libcxx/test/std/containers/associative/set/empty.fail.cpp
    M libcxx/test/std/containers/container.adaptors/priority.queue/priqueue.members/empty.fail.cpp
    M libcxx/test/std/containers/container.adaptors/queue/queue.defn/empty.fail.cpp
    M libcxx/test/std/containers/container.adaptors/stack/stack.defn/empty.fail.cpp
    M libcxx/test/std/containers/sequences/array/empty.fail.cpp
    M libcxx/test/std/containers/sequences/deque/deque.capacity/empty.fail.cpp
    M libcxx/test/std/containers/sequences/forwardlist/empty.fail.cpp
    M libcxx/test/std/containers/sequences/list/list.capacity/empty.fail.cpp
    M libcxx/test/std/containers/sequences/vector.bool/empty.fail.cpp
    M libcxx/test/std/containers/sequences/vector/vector.capacity/empty.fail.cpp
    M libcxx/test/std/containers/unord/unord.map/empty.fail.cpp
    M libcxx/test/std/containers/unord/unord.multimap/empty.fail.cpp
    M libcxx/test/std/containers/unord/unord.multiset/empty.fail.cpp
    M libcxx/test/std/containers/unord/unord.set/empty.fail.cpp
    M libcxx/test/std/input.output/filesystems/class.path/path.member/path.decompose/empty.fail.cpp
    M libcxx/test/std/iterators/iterator.container/empty.array.fail.cpp
    M libcxx/test/std/iterators/iterator.container/empty.container.fail.cpp
    M libcxx/test/std/iterators/iterator.container/empty.initializer_list.fail.cpp
    M libcxx/test/std/language.support/support.dynamic/new.delete/new.delete.placement/new_array_ptr.fail.cpp
    M libcxx/test/std/language.support/support.dynamic/new.delete/new.delete.placement/new_ptr.fail.cpp
    M libcxx/test/std/language.support/support.dynamic/new.delete/new.delete.single/new_size.fail.cpp
    M libcxx/test/std/language.support/support.dynamic/new.delete/new.delete.single/new_size_nothrow.fail.cpp
    M libcxx/test/std/language.support/support.dynamic/ptr.launder/launder.nodiscard.fail.cpp
    M libcxx/test/std/re/re.results/re.results.size/empty.fail.cpp
    M libcxx/test/std/strings/basic.string/string.capacity/empty.fail.cpp
    M libcxx/test/std/strings/string.view/string.view.capacity/empty.fail.cpp
    M libcxx/test/std/thread/futures/futures.async/async.fail.cpp
    M libcxx/test/std/utilities/allocator.adaptor/allocator.adaptor.members/allocate_size.fail.cpp
    M libcxx/test/std/utilities/allocator.adaptor/allocator.adaptor.members/allocate_size_hint.fail.cpp
    M libcxx/test/std/utilities/memory/allocator.traits/allocator.traits.members/allocate.fail.cpp
    M libcxx/test/std/utilities/memory/default.allocator/allocator.members/allocate.fail.cpp
    M libcxx/utils/libcxx/test/format.py

  Log Message:
  -----------
  [libc++] Require the use of clang-verify in .fail.cpp tests that don't fail without it

Some tests do not fail at all when -verify is not supported, unless some
arbitrary warning flag is added to make them fail. We currently used
-Werror=unused-result to make them fail, but doing so makes the test
suite a lot more inscrutable. It seems better to just disable those
tests when -verify is not supported.

Differential Revision: https://reviews.llvm.org/D76256