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

Siva Chandra via Phabricator via libc-commits libc-commits at lists.llvm.org
Wed Jul 29 23:14:44 PDT 2020


sivachandra added a comment.

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.


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