[all-commits] [llvm/llvm-project] f97936: [libc++] Cleanup and enable multiple warnings.

[Eric via All-commits]

  Branch: refs/heads/master
  Home:   https://github.com/llvm/llvm-project
  Commit: f97936fabd263e3b311e3b8e9ffca4920e4fcff0
      https://github.com/llvm/llvm-project/commit/f97936fabd263e3b311e3b8e9ffca4920e4fcff0
  Author: Eric Fiselier <eric at efcs.ca>
  Date:   2019-12-12 (Thu, 12 Dec 2019)

  Changed paths:
    M libcxx/include/__bit_reference
    M libcxx/include/__hash_table
    M libcxx/include/__tree
    M libcxx/include/ext/hash_map
    M libcxx/include/random
    M libcxx/include/valarray
    M libcxx/test/libcxx/containers/sequences/vector/asan_throw.pass.cpp
    M libcxx/test/std/utilities/meta/meta.const.eval/is_constant_evaluated.fail.cpp
    M libcxx/test/std/utilities/meta/meta.const.eval/is_constant_evaluated.pass.cpp
    M libcxx/test/std/utilities/optional/optional.object/optional.object.assign/copy.pass.cpp
    M libcxx/test/support/test_macros.h
    M libcxx/utils/libcxx/test/config.py

  Log Message:
  -----------
  [libc++] Cleanup and enable multiple warnings.

Too many warnings are being disabled too quickly. Warnings are
important to keeping libc++ correct. This patch re-enables two
warnings: -Wconstant-evaluated and -Wdeprecated-copy.

In future, all warnings disabled for the test suite should require
an attached bug. The bug should state the plan for re-enabling that
warning, or a strong case why it should remain disabled.