-
Notifications
You must be signed in to change notification settings - Fork 5.6k
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]Support build_cinn_pass in PIR+CINN #57779
Conversation
你的PR提交成功,感谢你对开源项目的贡献! |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
* [PIR]Support build_cinn_pass in PIR+CINN * fix conflict * fix .gitignore * polish cmakelists.txt * fix WITH_CINN * fix conflict * fix CmakeLists.txt
* [PIR]Support build_cinn_pass in PIR+CINN * fix conflict * fix .gitignore * polish cmakelists.txt * fix WITH_CINN * fix conflict * fix CmakeLists.txt
* [PIR]Support build_cinn_pass in PIR+CINN * fix conflict * fix .gitignore * polish cmakelists.txt * fix WITH_CINN * fix conflict * fix CmakeLists.txt
PR types
New features
PR changes
Others
Description
What's New?
Pcard-67164
see: #56880
将旧 Program+Graph 体系下的 build_cinn_pass 迁移至 pir::Pass 体系下,添加了相关单测。同时给基础组件新增了必要功能:
pir::Block
添加了Take(Op)
私有方法,用于转移 Op 的 ownership 到其他 Blockpir::Operation
添加了MoveTo(Block*, position)
方法,用于将std::vector<Operation*> ops
移动到cinn::GroupOp
Value
添加了operator <
的方法,用于std::set
的容器支持,方便进行差集算法操作BuildCinnPass
继承自pir::Pass
体系相关comment已在#57829 fix