Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Support inplace add #5432

Merged
merged 12 commits into from
Jul 10, 2021
Merged

Support inplace add #5432

merged 12 commits into from
Jul 10, 2021

Conversation

wyg1997
Copy link
Contributor

@wyg1997 wyg1997 commented Jul 8, 2021

image

@wyg1997 wyg1997 requested a review from hjchen2 July 8, 2021 12:07
@wyg1997 wyg1997 requested review from oneflow-ci-bot and removed request for oneflow-ci-bot July 10, 2021 02:20
@oneflow-ci-bot oneflow-ci-bot removed their request for review July 10, 2021 03:59
@oneflow-ci-bot oneflow-ci-bot self-requested a review July 10, 2021 04:01
@wyg1997 wyg1997 requested review from oneflow-ci-bot and removed request for oneflow-ci-bot July 10, 2021 05:15
@oneflow-ci-bot oneflow-ci-bot requested review from oneflow-ci-bot and removed request for oneflow-ci-bot July 10, 2021 05:47
@oneflow-ci-bot oneflow-ci-bot merged commit 3f52f24 into master Jul 10, 2021
@oneflow-ci-bot oneflow-ci-bot deleted the feat-add_op_inplace branch July 10, 2021 07:10
@wyg1997
Copy link
Contributor Author

wyg1997 commented Jul 21, 2021

BroadcaseAdd操作不应该支持inplace版本,而是先用BroadcaseLike做广播,再用inplace版本的ElementwiseAdd

#5551

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

Successfully merging this pull request may close these issues.

3 participants