-
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
Unify operator yamls directory #64446
Unify operator yamls directory #64446
Conversation
你的PR提交成功,感谢你对开源项目的贡献! |
个人的建议,可以酌情采纳:(1)这个 new 目录是不是直接叫 inconsistent_ops 更直观一些?(2) onednn.yaml 应该是只有 pir 静态图特有的吧,是不是叫 static_onednn.yaml更直观? |
… unify_ops_yaml_directory
… unify_ops_yaml_directory
已处理,new 目录改为了 inconsistent,onednn.yaml 改为了 onednn_static,yaml |
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.
Great work!
… unify_ops_yaml_directory
5a612c8
… unify_ops_yaml_directory
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
… unify_ops_yaml_directory
506fad1
PR Category
Others
PR Types
Others
Description
其中
paddle/phi/ops/yaml
目录下的是动静一致的算子,inconsistent
子目录下的是新 IR 下动静不一致的算子 (其中 rename 了几个 yaml 文件方便理解),legacy
子目录下则是面向旧 IR 静态图的算子,后续会随着旧 IR 进行退场