[PATCH] D104904: [OpenMP][AMDGCN] Initial math headers support
Jon Chesterfield via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Wed Jun 30 01:50:46 PDT 2021
JonChesterfield added inline comments.
================
Comment at: clang/test/Headers/Inputs/include/cstdlib:29
float fabs(float __x) { return __builtin_fabs(__x); }
+#endif
----------------
jdoerfert wrote:
> That seems to be fundamentally broken then, but let's see, maybe it will somehow work anyway.
I thought fabs was in math, not stdlib. Not sure what this file is doing but the functions above are inline and fabs isn't
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D104904/new/
https://reviews.llvm.org/D104904
More information about the cfe-commits
mailing list