-
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
[Prim][PIR] silu prim sink #59306
[Prim][PIR] silu prim sink #59306
Conversation
你的PR提交成功,感谢你对开源项目的贡献! |
auto one = full<T>(phi::vectorize(x.dims()), 1, x_tmp.dtype()); | ||
auto exp_temp = | ||
exp<T>(full<T>(phi::vectorize(x.dims()), -1, x_tmp.dtype()) * x_tmp); | ||
auto res = divide<T>(x_tmp, exp_temp + one); |
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.
suggest symbol overload
auto x_tmp = x; | ||
|
||
bool need_cast = | ||
org_dtype == phi::DataType::FLOAT16 || org_dtype == phi::DataType::UINT16; |
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.
suggest provide common utils like is_half_dtype(...)
auto x_tmp = x; | ||
|
||
bool need_cast = | ||
org_dtype == phi::DataType::FLOAT16 || org_dtype == phi::DataType::UINT16; |
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.
why unint16 instead of BFLOAT16?
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
* prim silu sink * prim silu sink * add is_half_dtype function
PR types
New features
PR changes
Others
Description
silu
op sink c++is_half_dtype()
function inutils.h
mean
、silu
、sqrt
、batch_norm
、softmax
、layer_norm
operator dtype judgment method