Skip to content

【CUDA Kernel No.65】Include kldiv_loss_kernel.h #2197

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Merged
merged 2 commits into from
Nov 24, 2025
Merged

【CUDA Kernel No.65】Include kldiv_loss_kernel.h #2197

merged 2 commits into from
Nov 24, 2025
+3 −2

Conversation

@algorithm1832
Copy link
Contributor

@algorithm1832 algorithm1832 commented Nov 21, 2025 •
edited
Loading

PR Category

Custom Device

PR Types

Improvements

Description

  • Include kldiv_loss_kernel.h to replace .cu file include
  • Add kldiv_loss_kernel.cu to metax_gpu's cmakelists

ℹ️The .h file already exists in the Paddle repo: paddle/phi/kernels/kldiv_loss_kernel.h

ℹ️The kldiv_loss_kernel.cu file has been included in iluvatar_gpu's cmakelists starting from line 276.

file(
  GLOB
  CUDA_SRCS2
  ...
  ${PADDLE_SOURCE_DIR}/paddle/phi/kernels/gpu/kldiv_loss_kernel.cu
  ...
)
...
set(CUDA_SRCS ${CUDA_SRCS1} ${CUDA_SRCS2})
list(REMOVE_DUPLICATES CUDA_SRCS)

Used AI Studio

@paddle-bot
Copy link

paddle-bot bot commented Nov 21, 2025

Thanks for your contribution!

@algorithm1832 algorithm1832 mentioned this pull request Nov 21, 2025
Open
23 tasks
@luotao1 luotao1 mentioned this pull request Nov 24, 2025
Open
@luotao1 luotao1 added the HappyOpenSource 快乐开源活动issue与PR label Nov 24, 2025
YqGe585
YqGe585 approved these changes Nov 24, 2025
View reviewed changes
Copy link
Collaborator

@YqGe585 YqGe585 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@YqGe585 YqGe585 merged commit 2ba4614 into PaddlePaddle:develop Nov 24, 2025
13 of 14 checks passed
@algorithm1832 algorithm1832 mentioned this pull request Nov 24, 2025
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@YqGe585 YqGe585 YqGe585 approved these changes

Assignees

@luotao1 luotao1

@YqGe585 YqGe585

Labels

contributor HappyOpenSource 快乐开源活动issue与PR

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@algorithm1832 @YqGe585 @luotao1