Skip to content

Conversation

@huangjiyi
Copy link
Member

@huangjiyi huangjiyi commented Aug 13, 2024

PR Category

CINN

PR Types

Bug fixes

Description

Pcard-76996

  • Fix ReduceTree Fusion

@paddle-bot
Copy link

paddle-bot bot commented Aug 13, 2024

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

1,
::common::errors::InvalidArgument(
"Downstream %s must have only one fusion_op.", instr->downstream_));
auto upstream_op = std::get<ReduceOp>(
Copy link
Contributor

Choose a reason for hiding this comment

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

加一个单侧。

Copy link
Member Author

Choose a reason for hiding this comment

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

下个 PR 完成

Comment on lines +124 to +133
std::string new_tmp_id() const {
if (cur_id_ == id_) {
cur_id_ = id_ + "_tmp_0";
} else {
int ith = std::stoi(cur_id_.substr(cur_id_.size() - 1));
cur_id_ = id_ + "_tmp_" + std::to_string(ith + 1);
}
return cur_id_;
}

Copy link
Contributor

Choose a reason for hiding this comment

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

这个不要作为成员函数吧,可以放到 utils 中作为一个工具函数。

Copy link
Member Author

Choose a reason for hiding this comment

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

下个 PR 完成

Copy link
Contributor

@2742195759 2742195759 left a comment

Choose a reason for hiding this comment

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

LGTM

@2742195759 2742195759 merged commit 0c34b1f into PaddlePaddle:develop Aug 14, 2024
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