[libc-commits] [libc] [libc] Fix linking of AMDGPU device runtime control constants for math (PR #65676)

Joseph Huber via libc-commits libc-commits at lists.llvm.org
Sat Sep 9 04:58:35 PDT 2023


jhuber6 wrote:

> The wavesize incompatibility is deeper than depends on the feature. It's a full ABI incompatibility and really should have 2 separate builds.
> 
> For the other math options, I have all the patches to eliminate them stuck in review

So, this approach is mostly just a quick solution to port the `ocml` libraries to this multi-architecture build I'm doing in `libc`. I think long-term I'd really like to move the builds to be common, since there's generally only a few builtins we could probably just make errors in the backend if they're unsupported once the user provides an architecture. The wavesize attribute is definitely the hardest one, but at least for my purposes we may be able to allocate 64 sized buffers always and use the intrinsic to set the upper limit.

https://github.com/llvm/llvm-project/pull/65676


More information about the libc-commits mailing list