[libc-commits] [PATCH] D84725: [libc] Add ULP function to MPFRNumber class to test correctly rounded functions such as SQRT, FMA.

Tue Ly via Phabricator via libc-commits libc-commits at lists.llvm.org
Wed Aug 5 12:16:42 PDT 2020


lntue added a comment.

In D84725#2183801 <https://reviews.llvm.org/D84725#2183801>, @sivachandra wrote:

> We currently have UnitTest depend on FPUtil. I think that an creates unnecessary dependency for tests which do not need FPUtil for testing. So, a plan could be:
>
> Instead of `EXPECT_EQ` and friends for floating point numbers, we will decouple UnitTest from FPUtil using matchers implemented in FPUtil. Say, `EXPECT_FP_MATCH` etc.
>
> WDYT? If you concur, then you need not do the work. I will do it, and you can absorb those changes into this patch.

That sounds good to me.  Let me know when it's ready.  Thanks!


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D84725



More information about the libc-commits mailing list