[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