[PATCH] D74275: [compiler-rt] Compile __powitf2 under wasm
Sam Clegg via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Mon Feb 10 19:52:08 PST 2020
sbc100 added inline comments.
================
Comment at: compiler-rt/lib/builtins/CMakeLists.txt:147
powitf2.c
+ powixf2.c
subdf3.c
----------------
efriedma wrote:
> powixf2.c is, in fact, x86-specific. At first glance it might not look like that, but the "x" in the name powixf2 refers exclusively to x86 long double.
Indeed, sorry I misread/misunderstood.
================
Comment at: compiler-rt/lib/builtins/powitf2.c:15
-#if _ARCH_PPC
+#if defined(_ARCH_PPC) || defined(__wasm__)
----------------
efriedma wrote:
> Please change this so it matches the other 128 long double routines: it should include fp_lib.h, it should use the condition `#if defined(CRT_HAS_128BIT) && defined(CRT_LDBL_128BIT)`, and it should use fp_t instead of explicitly writing out "long double".
Great those macros looks like what I'm looking for.
Would you mind doing the fp_t change as a followup? I don't know this codebase well enough to feel confident doing that part.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D74275/new/
https://reviews.llvm.org/D74275
More information about the llvm-commits
mailing list