[libc-commits] [PATCH] D143913: [libc][NFC] Make tuning macros start with LIBC_COPT_

Tue Ly via Phabricator via libc-commits libc-commits at lists.llvm.org
Mon Feb 13 10:20:27 PST 2023


lntue added inline comments.


================
Comment at: libc/test/src/math/RoundToIntegerTest.h:89
+    // is not forwarded to C++.
 #if LLVM_LIBC_IMPLEMENTATION_DEFINED_TEST_BEHAVIOR
     // Result is not well-defined, we always returns INTEGER_MAX
----------------
gchatelet wrote:
> @lntue I'm unsure what this is about so I didn't rename it. Is it supposed to be exposed at some point?
> There is a [CMake option here](https://github.com/llvm/llvm-project/blob/95526930da6c00f6f12816d1f99bdf99980c387f/libc/CMakeLists.txt#L68) but it is not sufficient to pass the option down to the C++ code.
I think we planned to have a section on the documentations about all the implementation-defined cases where we are different from glibc and maybe provide these flags for users.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D143913



More information about the libc-commits mailing list