Skip to content

Conversation

@FlyingQianMM
Copy link
Contributor

PR types

Bug fixes

PR changes

OPs

Describe

For index_select op, the function index_select_grad_init will not work on part of data whose number exceeds the loop number, so we add a inner loop by replacing for() with CUDA_KERNEL_LOOP.

@paddle-bot-old
Copy link

paddle-bot-old bot commented Apr 8, 2022

你的PR提交成功,感谢你对开源项目的贡献!
请关注后续CI自动化测试结果,详情请参考Paddle-CI手册
Your PR has been submitted. Thanks for your contribution!
Please wait for the result of CI firstly. See [Paddle CI Manual(https://www.paddlepaddle.org.cn/documentation/docs/zh/develop/dev_guides/git_guides/paddle_ci_manual_cn.html) for details.

@FlyingQianMM FlyingQianMM changed the title replace for with CUDA_KERNEL_LOOP for index_select_grad_init() in index_select op add a inner loop for index_select_grad_init() in index_select op when dealing with large-shape data Apr 8, 2022
ZzSean
ZzSean previously approved these changes Apr 8, 2022
Copy link
Contributor

@ZzSean ZzSean left a comment

Choose a reason for hiding this comment

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

LGTM

@FlyingQianMM FlyingQianMM merged commit bc01242 into PaddlePaddle:develop Apr 12, 2022
FlyingQianMM added a commit to FlyingQianMM/Paddle that referenced this pull request Apr 12, 2022
XiaoguangHu01 pushed a commit that referenced this pull request Apr 13, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

3 participants