[all-commits] [llvm/llvm-project] 537d90: [libc++] Split re.alg tests into locale-dependent ...

Alexander Richardson via All-commits all-commits at lists.llvm.org
Wed Jan 20 07:54:26 PST 2021


  Branch: refs/heads/master
  Home:   https://github.com/llvm/llvm-project
  Commit: 537d90db827d1df0fef400653eefd857834ca0ba
      https://github.com/llvm/llvm-project/commit/537d90db827d1df0fef400653eefd857834ca0ba
  Author: Alex Richardson <Alexander.Richardson at cl.cam.ac.uk>
  Date:   2021-01-20 (Wed, 20 Jan 2021)

  Changed paths:
    A libcxx/test/std/re/re.alg/re.alg.match/awk.locale.pass.cpp
    M libcxx/test/std/re/re.alg/re.alg.match/awk.pass.cpp
    A libcxx/test/std/re/re.alg/re.alg.match/basic.locale.pass.cpp
    M libcxx/test/std/re/re.alg/re.alg.match/basic.pass.cpp
    A libcxx/test/std/re/re.alg/re.alg.match/ecma.locale.pass.cpp
    M libcxx/test/std/re/re.alg/re.alg.match/ecma.pass.cpp
    A libcxx/test/std/re/re.alg/re.alg.match/extended.locale.pass.cpp
    M libcxx/test/std/re/re.alg/re.alg.match/extended.pass.cpp
    A libcxx/test/std/re/re.alg/re.alg.search/awk.locale.pass.cpp
    M libcxx/test/std/re/re.alg/re.alg.search/awk.pass.cpp
    A libcxx/test/std/re/re.alg/re.alg.search/basic.locale.pass.cpp
    M libcxx/test/std/re/re.alg/re.alg.search/basic.pass.cpp
    A libcxx/test/std/re/re.alg/re.alg.search/ecma.locale.pass.cpp
    M libcxx/test/std/re/re.alg/re.alg.search/ecma.pass.cpp
    A libcxx/test/std/re/re.alg/re.alg.search/extended.locale.pass.cpp
    M libcxx/test/std/re/re.alg/re.alg.search/extended.pass.cpp
    M libcxx/utils/libcxx/test/features.py

  Log Message:
  -----------
  [libc++] Split re.alg tests into locale-dependent and independent tests

Currently all these tests are XFAILED on Linux even though the problem
only seems to be with the few checks that look at collation. To retain
test coverage this splits the locale-dependent tests into a separate
.pass.cpp that is XFAILed as before.
This commit also XFAILs the locale-dependent tests on FreeBSD since the
[=M=] and [.ch.] behaviour for cs_CZ also doesn't seem to match the
behaviour that is expected by these tests.

Reviewed By: #libc, ldionne

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




More information about the All-commits mailing list