[libcxx-commits] [PATCH] D155312: [libc++] Use _Complex for multiplication and division of complex floating point types

Mark de Wever via Phabricator via libcxx-commits libcxx-commits at lists.llvm.org
Mon Jul 31 09:19:05 PDT 2023


Mordante added a comment.

Can you modify the CI file to run with a limited number of configurations, but enable all platform.

Also fix the non floating point test to be fixed.

That way I can get a better view of the impact of the changes.



================
Comment at: libcxx/test/std/numerics/complex.number/complex.member.ops/divide_equal_complex.pass.cpp:28
     c /= c2;
-    assert(c.real() == 1.5);
-    assert(c.imag() == 2.5);
+    assert(c.real() >= T(1.499999999999999));
+    assert(c.real() <= T(1.500000000000001));
----------------
I would like to test with an epsion and not `>=`.
I think the values used in the test should give an exact result.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D155312



More information about the libcxx-commits mailing list