arsenm added a comment.
Herald added subscribers: jplehr, sunshaoce.

In D139730#3991628 <https://reviews.llvm.org/D139730#3991628>, @JonChesterfield 
wrote:

> We currently require that library for libm, which I'm also not thrilled 
> about, but at least you can currently build and run openmp programs (that 
> don't use libm, like much of our tests) without it.

The ABI isn't defined in terms of what device-libs does. It's fixed offsets off 
of pointers accessible through amdgcn intrinsics. You can also just directly 
emit the same IR, these functions aren't complicated


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D139730/new/

https://reviews.llvm.org/D139730

_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
  • [PATCH] D139730: [OpenMP][D... Matt Arsenault via Phabricator via cfe-commits

Reply via email to