[libc] Enable MPFR library for math functions test

[hedingarcia]

Summary

Included more math functions to Windows's entrypoints
and made a cmake option (-DLLVM_LIBC_MPFR_INSTALL_PATH)
where the user can specify the install path where the MPFR
library was built so it can be linked. The try_compile was
moved to LLVMLibCCheckMPFR.cmake, so the variable that is
set after this process can retain its value in other files
of the same parent file. A direct reason for this is for
LIBC_TESTS_CAN_USE_MPFR to be true when the user specifies
MPFR's path and retain its value even after leaving the file.

[aeubanks]

libc/cmake/modules/LLVMLibCCheckMPFR.cmake

2

can you add an extra line after this?
and also at the very end of this file

4

just curious, what was the issue and fix to make ${LIBC_TESTS_CAN_USE_MPFR} work?

libc/utils/MPFRWrapper/CMakeLists.txt

13

message

[hedingarcia]

[libc] Added newline at end of files

[hedingarcia]

[libc] Fixed message statement

[hedingarcia]

libc/cmake/modules/LLVMLibCCheckMPFR.cmake

4

The issue with it was that once set to true, if the path to MPFR was given, that value was not saved and evaluated False in libc/test/src/CMakeLists.txt and messages displayed that "WARNING: Math test...will be skipped as MPFR library is not available"; the test that needed MPFR did not ran. A way around it was to set it to the value in a cmake file separately and that variable will be carried to the subdirectorys utils and test. Hence carrying the True value in both the CMakeLists.txt in MPFRWrapper and on the one in test/src. Value was not being shared between two CMakeLists.txt siblings but they were passed from parent to child.

[sivachandra]

libc/cmake/modules/LLVMLibCCheckMPFR.cmake

1

... where MPFR _is_ installed ... ?

Also, it would be nice if you can add something about this flag to step #9 here: https://github.com/llvm/llvm-project/tree/main/libc/config/windows#readme

[hedingarcia]

[libc] Fixed the summary of LLVM_LIBC_MPFR_INSTALL_PATH and explained this option in README.md

[sivachandra]

A suggestion on the wording but good to go.

libc/config/windows/README.md

65 ↗

(On Diff #362144)

You don't have line breaks in this big chunk of text! I have a suggestion wrt the wording:

Some LLVM libc math unittests test correctness/accuracy against results from
the [GNU MPFR library](https://www.mpfr.org/). If you want to run math tests
which use MPFR, and if MPFR on your machine is not installed in the default
include and linker lookup directories, then you can specify the MPFR install
directory by passing an additional CMake option as follows:

-DLLVM_LIBC_MPFR_INSTALL_PATH=<path/mpfr/install/dir>

If the above option is specified, then `${LLVM_LIBC_MPFR_INSTALL_PATH}/include`
will be added to the include directories, and
`${LLVM_LIBC_MPFR_INSTALL_PATH}/lib` will be added to the linker lookup
directories.

NOTE: The GNU MPFR library depends on the
[GNU GMP library](https://gmplib.org/). If you specify the above option, then it
will be assumed that GMP is also installed in the same directory or availabe in
the default paths.

[hedingarcia]

[libc] Changed the description in README.md to suggested message

[hedingarcia]

libc/config/windows/README.md

65 ↗

(On Diff #362144)

I agree, the message conveys more clearly the prerequisites for this CMake option.

libc/CMakeLists.txt

 # Path libc/scripts directory.
 set(LIBC_BUILD_SCRIPTS_DIR "${LIBC_SOURCE_DIR}/utils/build_scripts")
 
 set(LIBC_TARGET_OS ${CMAKE_SYSTEM_NAME})
 string(TOLOWER ${LIBC_TARGET_OS} LIBC_TARGET_OS)
 
 # Defines LIBC_TARGET_ARCHITECTURE and associated macros.
 include(LLVMLibCArchitectures)
+include(LLVMLibCCheckMPFR)
 
 # Flags to pass down to the compiler while building the libc functions.
 set(LIBC_COMPILE_OPTIONS_DEFAULT "" CACHE STRING "Architecture to tell clang to optimize for (e.g. -march=... or -mcpu=...)")
 
 # Check --print-resource-dir to find the compiler resource dir if this flag
 # is supported by the compiler.
 execute_process(
   OUTPUT_STRIP_TRAILING_WHITESPACE

libc/cmake/modules/LLVMLibCCheckMPFR.cmake

+set(LLVM_LIBC_MPFR_INSTALL_PATH "" CACHE PATH "Path to where MPFR was installed (e.g. C:/src/install or ~/src/install)")
+
+if(LLVM_LIBC_MPFR_INSTALL_PATH)
+  set(LIBC_TESTS_CAN_USE_MPFR TRUE)
+else()
+  try_compile(
+    LIBC_TESTS_CAN_USE_MPFR
+    ${CMAKE_CURRENT_BINARY_DIR}
+    SOURCES
+    ${LIBC_SOURCE_DIR}/utils/MPFRWrapper/check_mpfr.cpp
+    LINK_LIBRARIES
+      -lmpfr -lgmp
+  )
+endif()

libc/config/windows/entrypoints.txt

set(TARGET_LIBC_ENTRYPOINTS
     libc.src.string.strtok_r
 )
 
 set(TARGET_LIBM_ENTRYPOINTS
     # math.h entrypoints
     libc.src.math.copysign
     libc.src.math.copysignf
     libc.src.math.copysignl
-    libc.src.math.fdimf
+    libc.src.math.ceil
+    libc.src.math.ceilf
+    libc.src.math.ceill
+    libc.src.math.fabs
+    libc.src.math.fabsf
+    libc.src.math.fabsl
     libc.src.math.fdim
+    libc.src.math.fdimf
     libc.src.math.fdiml
+    libc.src.math.floor
+    libc.src.math.floorf
+    libc.src.math.floorl
+    libc.src.math.fma
+    libc.src.math.fmaf
     libc.src.math.fmin
     libc.src.math.fminf
     libc.src.math.fminl
     libc.src.math.fmax
     libc.src.math.fmaxf
     libc.src.math.fmaxl
+    libc.src.math.frexp
+    libc.src.math.frexpf
+    libc.src.math.frexpl
+    libc.src.math.hypot
+    libc.src.math.hypotf
     libc.src.math.ilogb
     libc.src.math.ilogbf
     libc.src.math.ilogbl
     libc.src.math.ldexp
     libc.src.math.ldexpf
     libc.src.math.ldexpl
+    libc.src.math.llround
+    libc.src.math.llroundf
+    libc.src.math.llroundl
     libc.src.math.logb
     libc.src.math.logbf
-    libc.src.math.logbl
+    libc.src.math.logbl 
+    libc.src.math.lround
+    libc.src.math.lroundf
+    libc.src.math.lroundl
     libc.src.math.modf
     libc.src.math.modff
     libc.src.math.modfl
+    libc.src.math.nearbyint
+    libc.src.math.nearbyintf
+    libc.src.math.nearbyintl
     libc.src.math.nextafter
     libc.src.math.nextafterf
     libc.src.math.nextafterl
+    libc.src.math.remainderf
+    libc.src.math.remainder
+    libc.src.math.remainderl
+    libc.src.math.remquof
+    libc.src.math.remquo
+    libc.src.math.remquol
+    libc.src.math.rint
+    libc.src.math.rintf
+    libc.src.math.rintl
+    libc.src.math.round
+    libc.src.math.roundf
+    libc.src.math.roundl
+    libc.src.math.sqrt
+    libc.src.math.sqrtf
+    libc.src.math.sqrtl
+    libc.src.math.trunc
+    libc.src.math.truncf
+    libc.src.math.truncl 
 )
 
 set(TARGET_LLVMLIBC_ENTRYPOINTS
   ${TARGET_LIBC_ENTRYPOINTS}
   ${TARGET_LIBM_ENTRYPOINTS}
 )

libc/utils/MPFRWrapper/CMakeLists.txt

-try_compile(
-  LIBC_TESTS_CAN_USE_MPFR
-  ${CMAKE_CURRENT_BINARY_DIR}
-  SOURCES
-    ${CMAKE_CURRENT_SOURCE_DIR}/check_mpfr.cpp
-  LINK_LIBRARIES
-    -lmpfr -lgmp
-)
-
 if(LIBC_TESTS_CAN_USE_MPFR)
   add_library(libcMPFRWrapper
     MPFRUtils.cpp
     MPFRUtils.h
   )
   add_dependencies(libcMPFRWrapper libc.utils.CPP.standalone_cpp libc.utils.FPUtil.fputil LibcUnitTest)
-  target_link_libraries(libcMPFRWrapper -lmpfr -lgmp LibcFPTestHelpers LibcUnitTest)
+  if(EXISTS ${LLVM_LIBC_MPFR_INSTALL_PATH})
+    target_include_directories(libcMPFRWrapper PUBLIC ${LLVM_LIBC_MPFR_INSTALL_PATH}/include)
+    target_link_directories(libcMPFRWrapper PUBLIC ${LLVM_LIBC_MPFR_INSTALL_PATH}/lib)
+  endif()
+  target_link_libraries(libcMPFRWrapper LibcFPTestHelpers LibcUnitTest mpfr gmp)
 else()
   message(WARNING "Math tests using MPFR will be skipped.")
 endif()