[libcxx-commits] [PATCH] D120448: [libc++][AIX][test] Enable put_double/long_double locale tests

[Jinsong Ji via Phabricator via libcxx-commits]

jsji added inline comments.


================
Comment at: libcxx/test/std/localization/locale.categories/category.numeric/locale.nm.put/facet.num.put.members/put_long_double.pass.cpp:26220
     test3();
+#ifndef _AIX
+    // AIX will print INF for inf and NaNQ for nan regardless of uppercase.
----------------
ldionne wrote:
> Can we instead fix those tests on AIX, i.e.
> 
> ```
> #ifdef _AIX
>   assert(something)
> #else
>   assert(something else)
> #endif
> ```
> 
Sure.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D120448/new/

https://reviews.llvm.org/D120448