[libc-commits] [PATCH] D86241: [libc] Add a class called NormalFloat which represents normalized floats.

Siva Chandra via Phabricator via libc-commits libc-commits at lists.llvm.org
Wed Aug 19 13:05:18 PDT 2020


sivachandra added a comment.

@lntue - I want to use this change for few followup implementations. Also, I see that you have normalizaton functions in  your sqrt patch wherein you use a binary search approach for normalization. I did a simple linear normalization in this patch. You can absorb this into your patch and implement the binary search normalization over it. I don't want to appear like I am stealing ;)

I want to avoid doing floating point multiplications so I will prefer your binary search normalization over the multiplication approach.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D86241



More information about the libc-commits mailing list