[libcxx] r278387 - Remove test for the sign of a NaN - doesn't work on MIPS, not strictly legal. Fixes bug 28936

Hans Wennborg via cfe-commits cfe-commits at lists.llvm.org
Thu Aug 11 14:55:36 PDT 2016


Merged to 3.9 in r278425, as the PR was marked as a release blocker.

On Thu, Aug 11, 2016 at 11:46 AM, Marshall Clow via cfe-commits
<cfe-commits at lists.llvm.org> wrote:
> Author: marshall
> Date: Thu Aug 11 13:46:24 2016
> New Revision: 278387
>
> URL: http://llvm.org/viewvc/llvm-project?rev=278387&view=rev
> Log:
> Remove test for the sign of a NaN - doesn't work on MIPS, not strictly legal. Fixes bug 28936
>
> Modified:
>     libcxx/trunk/test/std/numerics/complex.number/complex.transcendentals/asin.pass.cpp
>
> Modified: libcxx/trunk/test/std/numerics/complex.number/complex.transcendentals/asin.pass.cpp
> URL: http://llvm.org/viewvc/llvm-project/libcxx/trunk/test/std/numerics/complex.number/complex.transcendentals/asin.pass.cpp?rev=278387&r1=278386&r2=278387&view=diff
> ==============================================================================
> --- libcxx/trunk/test/std/numerics/complex.number/complex.transcendentals/asin.pass.cpp (original)
> +++ libcxx/trunk/test/std/numerics/complex.number/complex.transcendentals/asin.pass.cpp Thu Aug 11 13:46:24 2016
> @@ -91,7 +91,6 @@ void test_edges()
>          {
>              assert(std::isnan(r.real()));
>              assert(std::isnan(r.imag()));
> -            assert(std::signbit(testcases[i].imag()) == std::signbit(r.imag()));
>          }
>          else if (std::isnan(testcases[i].real()) && std::isinf(testcases[i].imag()))
>          {
>
>
> _______________________________________________
> cfe-commits mailing list
> cfe-commits at lists.llvm.org
> http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits


More information about the cfe-commits mailing list