jhuber6 added a comment. In D139730#4418630 <https://reviews.llvm.org/D139730#4418630>, @arsenm wrote:
> 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 This is the suggestion I've talked with @saiislam about. I think we should just copy the magic intrinsics that are being queried here. I'm assuming we don't need to bother with supporting both v4 and v5 so we can just make the switch all at once. 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
