[libclc] [libclc] Move __clc_ldexp to CLC library (PR #126078)

Fraser Cormack via cfe-commits cfe-commits at lists.llvm.org
Thu Feb 6 07:23:30 PST 2025


frasercrmck wrote:

> The amdgpu customization should be removed. This is legacy code from before we had the ldexp intrinsic. Regular __builtin_ldexp* should work now

Good to know, thanks. No one else in libclc is using `__builtin_ldexp`, so in its current form it'd still have to be some sort of a customization. Everyone is opting for the software implementation.

We could have `__builtin_ldexp*` in the default implementation, but guard it behind a macro that only AMD defines. Would we ideally have an `__builtin_elementwise_ldexp` to target the vector intrinsics directly?

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


More information about the cfe-commits mailing list