[all-commits] [llvm/llvm-project] 706e37: [libc++][AIX][test] Enable put_double/long_double ...

jsji via All-commits all-commits at lists.llvm.org
Mon Mar 7 11:21:49 PST 2022


  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: 706e37c8e0e3237b39972fa45953daedd6ec9696
      https://github.com/llvm/llvm-project/commit/706e37c8e0e3237b39972fa45953daedd6ec9696
  Author: Jinsong Ji <jji at us.ibm.com>
  Date:   2022-03-07 (Mon, 07 Mar 2022)

  Changed paths:
    M libcxx/test/std/localization/locale.categories/category.numeric/locale.nm.put/facet.num.put.members/put_long_double.pass.cpp

  Log Message:
  -----------
  [libc++][AIX][test] Enable put_double/long_double locale tests

AIX print -0.0 , inf, nan differently, which are causing the test
failures. We are OK for most other tests.

This patch remove the tests related these limitations conditionally on AIX,
so that we can enable the other tests to avoid losing test coverage.

The general direction is:

```
if strings don't differ between environments, keep the string literal "INF" and the padding, instead of folding them into variables.
```

Reviewed By: ldionne, #libc, Mordante

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




More information about the All-commits mailing list