[Openmp-commits] [PATCH] D139730: [OpenMP][DeviceRTL][AMDGPU] Support code object version 5

Joseph Huber via Phabricator via Openmp-commits openmp-commits at lists.llvm.org
Tue Jun 13 11:56:43 PDT 2023

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.

  rG LLVM Github Monorepo



More information about the Openmp-commits mailing list