[libc-commits] [PATCH] D115828: [libc] Implement correctly rounded log2f based on RLIBM library.

[Siva Chandra via Phabricator via libc-commits]

sivachandra added inline comments.


================
Comment at: libc/src/math/generic/common_constants.h:15
+#pragma clang diagnostic push
+#pragma clang diagnostic ignored "-Wc++17-extensions"
+
----------------
lntue wrote:
> michaelrj wrote:
> > is this intended to be a long term solution? It feels like there should be an easier way to handle using hexadecimal float constants.
> I think the long term solution is to build libc with C++17 standard and get rid of these.
Can you check if adding `-std=c++17` to COMPILE_OPTIONS fixes the warning problem for you?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D115828