[libc-commits] [PATCH] D124204: [libc] Support 32-bit ARM platform tests

[Dominic Chen via Phabricator via libc-commits]

ddcc added inline comments.


================
Comment at: libc/test/src/stdlib/strtold_test.cpp:18
 
+#ifdef __SIZEOF_INT128__
 class LlvmLibcStrToLDTest : public __llvm_libc::testing::Test {
----------------
michaelrj wrote:
> for these tests I would recommend disabling `strtold` as an entrypoint instead of putting the whole test file in an `#ifdef`. If you want, you can add a TODO for me to look into changing this test to not use uint128 when it doesn't need it.
Are you referring to libc/config/darwin/arm/entrypoints.txt? Or omitting strtold_tests from libc/test/src/stdlib/CMakeLists.txt for arm?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D124204