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

[cherry-pick] [ARM] Add int64 implement for gather and greater_than #4444

Merged
merged 2 commits into from
Sep 24, 2020

Conversation

DannyIsFunny
Copy link
Collaborator

@DannyIsFunny DannyIsFunny commented Sep 24, 2020

cherry-picked from: #4342 ,修复gcn模型不能运行问题
【背景】 gather 算子和 greater_than算子算子缺乏int64实现,Paddlepaddle中该算子有int64实现。 导致PaddlePaddle训练出的部分模型在Paddle-Lite上加载失败。

【本PR工作】为了保证PaddlePaddle训练出来的模型在Paddle-Lite上可以正常预测,本PR为gathergreater_than 补充了int64实现

@DannyIsFunny DannyIsFunny changed the title [cherry-pick] Fix the [cherry-pick] [ARM] Add int64 implement for gather and greater_than Sep 24, 2020
Copy link
Contributor

@ysh329 ysh329 left a comment

Choose a reason for hiding this comment

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

LGTM

Copy link
Collaborator

@chenjiaoAngel chenjiaoAngel left a comment

Choose a reason for hiding this comment

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

LGTM

Copy link
Collaborator

@zhaoyang-star zhaoyang-star left a comment

Choose a reason for hiding this comment

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

LGTM

Copy link
Collaborator

@hong19860320 hong19860320 left a comment

Choose a reason for hiding this comment

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

LGTM

@DannyIsFunny DannyIsFunny merged commit 4b5b540 into PaddlePaddle:release/v2.7 Sep 24, 2020
@DannyIsFunny DannyIsFunny deleted the cherry-gcn branch September 24, 2020 16:38
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.

7 participants