Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Conversation

@YqGe585
Copy link
Contributor

@YqGe585 YqGe585 commented Aug 25, 2025

PR Category

Custom Device

PR Types

Bug fixes

Description

pcard-67164
If PADDLE_WITH_CUDA and PADDLE_WITH_XCCL are both on, ncclSuccess should be supported.

@paddle-bot
Copy link

paddle-bot bot commented Aug 25, 2025

你的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 for details.


#if !defined(__APPLE__) && defined(PADDLE_WITH_NCCL)
#if !defined(__APPLE__) && \
(defined(PADDLE_WITH_NCCL) || defined(PADDLE_WITH_XCCL))
Copy link
Contributor

Choose a reason for hiding this comment

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

PADDLE_WITH_XCCL的时候打开如果是非cuda类的,会有问题吗

Copy link
Contributor Author

Choose a reason for hiding this comment

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

上面还有PADDLE_WITH_CUDA的宏来控制

Copy link
Contributor

@yongqiangma yongqiangma left a comment

Choose a reason for hiding this comment

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

LGTM

@yongqiangma yongqiangma merged commit 8d38442 into PaddlePaddle:develop Aug 29, 2025
72 of 74 checks passed
@YqGe585 YqGe585 deleted the fixenforce branch September 16, 2025 03:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants