[libc-commits] [PATCH] D91665: [libc] Make more of the libc unit testing llvm independent

[Siva Chandra via Phabricator via libc-commits]

sivachandra added inline comments.


================
Comment at: libc/utils/UnitTest/CMakeLists.txt:5
   Test.h
-  LINK_COMPONENTS Support
 )
----------------
You should probably do the same in `utils/testutils/CMakeLists.txt`? I will do another pass and switch these targets to `add_library` after I move FPUtil's `TestHelpers`.


================
Comment at: libc/utils/testutils/ExecuteFunctionUnix.cpp:12
 #include <cstdlib>
+#include <cstring>
+#include <iostream>
----------------
Why do we need `cstring`?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D91665