[llvm] [Reland][TLI] Add support for hypot libcall. (PR #114343)

Kenji Mouri / 毛利 研二 via llvm-commits llvm-commits at lists.llvm.org
Wed Oct 30 22:03:21 PDT 2024


================
@@ -300,6 +300,7 @@ static void initializeLibCalls(TargetLibraryInfoImpl &TLI, const Triple &T,
       TLI.setUnavailable(LibFunc_expf);
       TLI.setUnavailable(LibFunc_floorf);
       TLI.setUnavailable(LibFunc_fmodf);
+      TLI.setUnavailable(LibFunc_hypotf);
----------------
MouriNaruto wrote:

I also have the same question because I also confused for that. I finally chose that place because I followed https://github.com/llvm/llvm-project/pull/99611 (I'm sorry. It's my first time contributing to that. So I chose to follow other people's way.) 

According to https://en.cppreference.com/w/c/numeric/math/remquo, the remquo series functions seems introduced in C99 but this commit adds remquo to the C89 code block I added the hypot to.
https://github.com/llvm/llvm-project/blob/6effab990c5c1b4fe55fcd43004a1fd88145bb8d/llvm/lib/Analysis/TargetLibraryInfo.cpp#L291-L319

https://github.com/llvm/llvm-project/pull/114343


More information about the llvm-commits mailing list