[libc-commits] [PATCH] D129918: [libc] Add utility classes for checking exceptional values.

[Tue Ly via Phabricator via libc-commits]

lntue marked 2 inline comments as done.
lntue added inline comments.


================
Comment at: libc/src/__support/FPUtil/except_value_utils.h:54
+
+  static bool check(const ExceptionalValues &ExceptionalValues, UIntType x_bits,
+                    T &result) {
----------------
sivachandra wrote:
> Because of the name like `check`, I was thinking this is a test utility. May be call it `expect_value_output` or something like that?
> 
> Also, @jeffbailey is working on an equivalent of `std::optional`: https://reviews.llvm.org/D129920. It seems like using it here would be appropriate? 
> 
> Lastly, does it have to be a static method - can it not be a non-static method? Then, you can name the method just `get_output`?
I updated the patch to use `cpp::Optional` from @jeffbailey patch https://reviews.llvm.org/D129920, and change these to standalone methods. 


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D129918