[libc-commits] [PATCH] D79150: [libc] Add definitions of double_t and float_t to math.h.

Alex Brachet via Phabricator via libc-commits libc-commits at lists.llvm.org
Wed Apr 29 21:05:23 PDT 2020


abrachet added inline comments.


================
Comment at: libc/include/__llvm-libc-stdc-types.h:17
+#if !defined(__FLT_EVAL_METHOD__) || __FLT_EVAL_METHOD__ == 0
+#define __DEFINE_LLVM_LIBC_FLOAT_T typedef float float_t
+#define __DEFINE_LLVM_LIBC_DOUBLE_T typedef double double_t
----------------
Is it better to do `#define __LLVM_LIBC_FLOAT_T float` and then `typdef __LLVM_LIBC_FLOAT_T float_t` bellow? That also somewhat mirrors what we do in __posix_types.h with `__INT64_TYPE__`


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D79150





More information about the libc-commits mailing list