Skip to content

Conversation

@qili93
Copy link
Contributor

@qili93 qili93 commented Feb 20, 2021

PR types

New features

PR changes

Others

Describe

update fluid collective op for rocm

@paddle-bot-old
Copy link

Thanks for your contribution!
Please wait for the result of CI firstly. See Paddle CI Manual for details.

void Compute(const framework::ExecutionContext &ctx) const override {
#if defined(PADDLE_WITH_NCCL) && NCCL_VERSION_CODE >= 2703
#if (defined(PADDLE_WITH_RCCL) || defined(PADDLE_WITH_NCCL)) && \
NCCL_VERSION_CODE >= 2703
Copy link
Contributor

Choose a reason for hiding this comment

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

RCCL里用的是NCCL_VERSION_CODE还是RCCL_VERSION_CODE

Copy link
Contributor

Choose a reason for hiding this comment

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

NCCL_VERSION_CODE

Copy link
Contributor

Choose a reason for hiding this comment

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

可以加个注释说明一下,RCCL用的也是NCCL_VERSION_CODE,除了名字不一样,其余的基本一致。

Copy link
Contributor Author

Choose a reason for hiding this comment

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

RCCL里面用的也是NCCL_VERSION_CODE

Copy link
Contributor

@wangxicoding wangxicoding left a comment

Choose a reason for hiding this comment

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

LGTM

@qili93 qili93 requested a review from GaoWei8 February 24, 2021 02:49
Copy link
Contributor

@GaoWei8 GaoWei8 left a comment

Choose a reason for hiding this comment

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

LGTM for op benchmark

@qili93 qili93 merged commit ee76ea7 into PaddlePaddle:develop Feb 24, 2021
@qili93 qili93 deleted the rocm_collective_op branch February 24, 2021 04:21
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.

3 participants