Skip to content

Conversation

algorithm1832
Copy link
Contributor

PR Category

Custom Device

PR Types

Improvements

Description

  • Include c_embedding_grad_kernel.h to replace .cu file include

ℹ️The .h file already exists in the Paddle repo. So there's no modifications to the Paddle repo.

Copy link

paddle-bot bot commented Oct 3, 2025

Thanks for your contribution!

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.

需要在cmakelists中的编译列表中添加paddle/phi/kernels/gpu/c_embedding_grad_kernel.cu,否则会缺少函数实现。

@algorithm1832
Copy link
Contributor Author

经检查,c_embedding_grad_kernel.cu已包含在metax_gpu的CMakeLists.txt中(第150行),它被写入CUDA_SRCS

file(
  GLOB
  CUDA_SRCS
  ...
  ${PADDLE_SOURCE_DIR}/paddle/phi/kernels/gpu/c_embedding_grad_kernel.cu
  ...
)

后续进行Kernel规范化任务时,如CMakeLists中已包含,会进行说明。非常感谢!

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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

contributor HappyOpenSource 快乐开源活动issue与PR

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants