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

[PIR]Add cinn::dialect::GroupOp for BuildCinnPass #57304

Merged
merged 3 commits into from
Sep 19, 2023

Conversation

Aurelius84
Copy link
Contributor

@Aurelius84 Aurelius84 commented Sep 14, 2023

PR types

Others

PR changes

OPs

Description

Pcard-67164

see: #56880

整体适配思路详见技术文档:#56879

33cee7818a3186e6c9a618fedbcb9849

@paddle-bot
Copy link

paddle-bot bot commented Sep 14, 2023

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

// TODO(Aurelius84): Need to figure out what we need indeed for GroupOp.
// Currently we paste almost members here and will remove them step by
// step.
struct GroupInfo {
Copy link
Contributor Author

Choose a reason for hiding this comment

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

GroupInfo 目前在Compiler阶段以及Fusion Merge都会用到,目前是作为了一个Attribute把那定到Group里,方便存取。后续将考虑看是否可以解耦,以独立于pir::Op体系


// TODO(Aurlius84): Need to figure out how to rebuild relation info of ops outer
// GroupOp
void GroupOp::Build(pir::Builder &builder,
Copy link
Contributor Author

Choose a reason for hiding this comment

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

GroupOp的构建和内在Ops的「断边」以及「重建」将在另一个PR里的BuildCinnPass::ReplaceWithGroupOp 里实现

@Aurelius84 Aurelius84 changed the title [PIR]Add cinn::dialect::GroupOp [PIR]Add cinn::dialect::GroupOp for BuildCinnPass Sep 19, 2023
Copy link
Contributor

@winter-wang winter-wang 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
Contributor

@risemeup1 risemeup1 left a comment

Choose a reason for hiding this comment

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

LGTM

@Aurelius84 Aurelius84 merged commit 593e97e into PaddlePaddle:develop Sep 19, 2023
Frida-a pushed a commit to Frida-a/Paddle that referenced this pull request Oct 14, 2023
* [PIR]Add cinn::dialect::GroupOp

* fix unittest

* fix typo
danleifeng pushed a commit to danleifeng/Paddle that referenced this pull request Nov 14, 2023
* [PIR]Add cinn::dialect::GroupOp

* fix unittest

* fix typo
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.

4 participants