[all-commits] [llvm/llvm-project] b48a25: [libcxx] [test] Don't look for the literal spellin...

Martin Storsjö via All-commits all-commits at lists.llvm.org
Mon May 31 12:13:54 PDT 2021


  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: b48a256e9c6e300f9f3f20180db369b9e7e27250
      https://github.com/llvm/llvm-project/commit/b48a256e9c6e300f9f3f20180db369b9e7e27250
  Author: Martin Storsjö <martin at martin.st>
  Date:   2021-05-31 (Mon, 31 May 2021)

  Changed paths:
    M libcxx/test/std/localization/locale.categories/category.ctype/locale.codecvt.byname/codecvt_byname_char16_t_char.depr_in_cxx20.verify.cpp
    M libcxx/test/std/localization/locale.categories/category.ctype/locale.codecvt.byname/codecvt_byname_char32_t_char.depr_in_cxx20.verify.cpp
    M libcxx/test/std/localization/locale.categories/category.ctype/locale.codecvt/codecvt_char16_t_char.depr_in_cxx20.verify.cpp
    M libcxx/test/std/localization/locale.categories/category.ctype/locale.codecvt/codecvt_char32_t_char.depr_in_cxx20.verify.cpp

  Log Message:
  -----------
  [libcxx] [test] Don't look for the literal spelling __mbstate_t in warnings

While the code uses the type name `std::mbstate_t`, the warning message
mentions the original underlying type, which is a C library internal
type name.

On Windows this type is called `_Mbstatet` instead of `__mbstate_t`. Use
expect-warning-re to avoid spelling out the literal name of the type.

Due to issues with the detection of the clang-verify feature, these
tests have been skipped in the Windows CI configuration so far.

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


  Commit: c8644ae1e98c38e44460f50cf9eeba979b32a7e6
      https://github.com/llvm/llvm-project/commit/c8644ae1e98c38e44460f50cf9eeba979b32a7e6
  Author: Martin Storsjö <martin at martin.st>
  Date:   2021-05-31 (Mon, 31 May 2021)

  Changed paths:
    M libcxx/include/__config
    M libcxx/test/libcxx/language.support/has_timespec_get.compile.pass.cpp
    M libcxx/test/std/language.support/support.runtime/ctime.pass.cpp
    M libcxx/test/std/utilities/time/date.time/ctime.pass.cpp

  Log Message:
  -----------
  [libcxx] Define LIBCPP_HAS_TIMESPEC_GET for MSVC configurations

This define was out of sync with the corresponding define in tests, it
was added inconsistently in 171c77b7dad4a050aca23ff7ca1b75f87560fefb.

Modern MSVC environments do have these typedefs and functions.

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


  Commit: be34d5f74afe8347b8b6110cb209186caf07e16b
      https://github.com/llvm/llvm-project/commit/be34d5f74afe8347b8b6110cb209186caf07e16b
  Author: Martin Storsjö <martin at martin.st>
  Date:   2021-05-31 (Mon, 31 May 2021)

  Changed paths:
    M libcxx/test/libcxx/language.support/has_aligned_alloc.compile.pass.cpp
    M libcxx/test/std/depr/depr.c.headers/stdlib_h.pass.cpp
    M libcxx/test/std/language.support/support.runtime/cstdlib.pass.cpp
    M libcxx/test/support/test_macros.h

  Log Message:
  -----------
  [libcxx] [test] Remove an incorrect TEST_HAS_ALIGNED_ALLOC define from test_macros.h

This was added inconsistently in
19fd9039ca242f408493b5c662f9d908eab8555e; Windows doesn't have the
aligned_alloc function (neither MSVC nor MinGW toolchains) and we don't
define _LIBCPP_HAS_ALIGNED_ALLOC while building libcxx.

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


  Commit: 23f7f56cecf9015f5fbee5b502430fd6985d269a
      https://github.com/llvm/llvm-project/commit/23f7f56cecf9015f5fbee5b502430fd6985d269a
  Author: Martin Storsjö <martin at martin.st>
  Date:   2021-05-31 (Mon, 31 May 2021)

  Changed paths:
    M libcxx/test/libcxx/include_as_c.sh.cpp

  Log Message:
  -----------
  [libcxx] [test] Fix the include_as_c.sh.cpp test in MSVC configurations

Avoid including a header that is known not to work with clang in MSVC
mode when compiling as C.

(Alternatively, this could be something like "XFAIL: clang && msvc",
but I think it's more useful to actually check the rest of the test
instead of expecting the whole test to fail.)

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


Compare: https://github.com/llvm/llvm-project/compare/aa00b1d76364...23f7f56cecf9


More information about the All-commits mailing list