[clang] [HLSL] Implement the `ldexp` intrinsic (PR #138182)

Farzon Lotfi via cfe-commits cfe-commits at lists.llvm.org
Fri May 2 05:38:05 PDT 2025


https://github.com/farzonl approved this pull request.

This looks fine.

 I would not like to carry forward native half testing of  intrinsics that are written purely in HLSL. I think the builtin ones aren't great either but I understand how they came to exist.

As for looking at notes. its easy to mess up the the vector size enforcements. and we have been using notes to confirm we did them right. I know for some notes we are going to have a bunch of repeated things we don't care about. But There is benefit to keeping them as there is no other way to confirm that the `no matching function for call to 'ldexp'` is happeing because of vector size out of bounds.

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


More information about the cfe-commits mailing list