[clang] [CodeGenPrepare] Transform ldexp into target supported intrinsics (PR #67552)

2023-09-27 Thread via cfe-commits
huhu233 wrote: > Why does this need special handling in CGP instead of being a normal custom > lowering for FLDEXP? @nikic , thanks for your suggestion, I'll update the patch as soon! https://github.com/llvm/llvm-project/pull/67552 ___ cfe-commits

[clang] [CodeGenPrepare] Transform ldexp into target supported intrinsics (PR #67552)

2023-09-27 Thread via cfe-commits
huhu233 wrote: > I agree and custom lowering also gives a straight forward way to support > vectors types as well. You are right, I'll update the patch as soon, thanks! https://github.com/llvm/llvm-project/pull/67552 ___ cfe-commits mailing list

[clang] [CodeGenPrepare] Transform ldexp into target supported intrinsics (PR #67552)

2023-09-27 Thread Paul Walker via cfe-commits
paulwalker-arm wrote: I agree and custom lowering also gives a straight forward way to support vectors types as well. https://github.com/llvm/llvm-project/pull/67552 ___ cfe-commits mailing list cfe-commits@lists.llvm.org

[clang] [CodeGenPrepare] Transform ldexp into target supported intrinsics (PR #67552)

2023-09-27 Thread Nikita Popov via cfe-commits
nikic wrote: Why does this need special handling in CGP instead of being a normal custom lowering for FLDEXP? https://github.com/llvm/llvm-project/pull/67552 ___ cfe-commits mailing list cfe-commits@lists.llvm.org

[clang] [CodeGenPrepare] Transform ldexp into target supported intrinsics (PR #67552)

2023-09-27 Thread via cfe-commits
llvmbot wrote: @llvm/pr-subscribers-backend-aarch64 Changes There are more efficient implementations for llvm.ldexp on different targets. This patch transforms llvm.ldexp into target supported intrinsics before lowering. --- Full diff:

[clang] [CodeGenPrepare] Transform ldexp into target supported intrinsics (PR #67552)

2023-09-27 Thread via cfe-commits
https://github.com/huhu233 created https://github.com/llvm/llvm-project/pull/67552 There are more efficient implementations for llvm.ldexp on different targets. This patch transforms llvm.ldexp into target supported intrinsics before lowering. >From dce0b8f8a76da24b27dd6ed61087e97b5f9415ec