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

[CINN] 【Infer Symbolic Shape BUAA 】Add fused_feedfoward op #68183

Merged
merged 7 commits into from
Oct 11, 2024

Conversation

crazyxiaoxi
Copy link
Contributor

@crazyxiaoxi crazyxiaoxi commented Sep 12, 2024

PR Category

CINN

PR Types

improvements

Description

添加fused_feedfoward算子符号推导接口
已经包含Optest
Pcard-67164

first
Copy link

paddle-bot bot commented Sep 12, 2024

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

@luotao1 luotao1 added contributor External developers HappyOpenSource Pro 进阶版快乐开源活动,更具挑战性的任务 labels Sep 13, 2024
@crazyxiaoxi crazyxiaoxi changed the title [CINN] 【Infer Symbolic Shape BUAA 】Add fused_feedfoward op [CINN] 【Infer Symbolic Shape BUAA 】Add pool3d op Sep 14, 2024
@crazyxiaoxi crazyxiaoxi changed the title [CINN] 【Infer Symbolic Shape BUAA 】Add pool3d op [CINN] 【Infer Symbolic Shape BUAA 】Add fused_feedfoward op Sep 14, 2024
Copy link

paddle-ci-bot bot commented Sep 22, 2024

Sorry to inform you that 32afae7's CIs have passed for more than 7 days. To prevent PR conflicts, you need to re-run all CIs manually.

@@ -1011,7 +1011,7 @@
func: FusedFeedForwardInferMeta
optional: dropout1_seed, dropout2_seed, linear1_bias, linear2_bias, ln1_scale, ln1_bias, ln2_scale, ln2_bias, ln2_mean, ln2_variance, ln1_mean, ln1_variance, ln1_out
Copy link
Contributor

Choose a reason for hiding this comment

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

这几个optional类型的输出,set前也需要用isFakevalue判断下

Copy link
Contributor Author

Choose a reason for hiding this comment

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

已处理

@@ -18,6 +18,7 @@
#include "paddle/fluid/pir/dialect/operator/interface/infer_symbolic_shape/infer_sym_slice_utils.h"
#include "paddle/fluid/pir/dialect/operator/interface/infer_symbolic_shape/infer_sym_utils.h"
#include "paddle/fluid/pir/dialect/operator/ir/op_attribute.h"
#include "paddle/phi/kernels/funcs/blas/blas.h"
Copy link
Contributor

Choose a reason for hiding this comment

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

下面的代码中有用到blas.h里的接口吗

Copy link
Contributor

@gongshaotian gongshaotian left a comment

Choose a reason for hiding this comment

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

删掉blas的引用,优化下auto的用法

Copy link

paddle-ci-bot bot commented Oct 3, 2024

Sorry to inform you that 3022d5a's CIs have passed for more than 7 days. To prevent PR conflicts, you need to re-run all CIs manually.

@gongshaotian
Copy link
Contributor

LGTM

@luotao1 luotao1 merged commit d821135 into PaddlePaddle:develop Oct 11, 2024
26 of 27 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
contributor External developers HappyOpenSource Pro 进阶版快乐开源活动,更具挑战性的任务
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants