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

CI测试不review #64057

Closed
wants to merge 3 commits into from
Closed

CI测试不review #64057

wants to merge 3 commits into from

Conversation

co63oc
Copy link
Contributor

@co63oc co63oc commented May 6, 2024

PR Category

Others

PR Types

Others

Description

移除
paddle/fluid/operators/fused/attention_layer_norm.h

Copy link

paddle-bot bot commented May 6, 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.

@SigureMo
Copy link
Member

SigureMo commented May 6, 2024

替换头文件后 WARP_SIZE 提示错误,修改名称,如果不修改错误信息可以查看PR #64057

这个 @zrr1999 有经验,#61529 里也遇到过这个问题,报错看起来也是一样的(词法上的),应该是这里的 WARP_SIZE 与其他地方 define 的 WARP_SIZE 冲突,宏展开后 template <int WARP_SIZE> 变为 template <int 32> 导致词法分析时就出问题了,解决方案也很简单,要么改模板参数名字,要么改宏的名字

@co63oc
Copy link
Contributor Author

co63oc commented May 6, 2024

替换头文件后 WARP_SIZE 提示错误,修改名称,如果不修改错误信息可以查看PR #64057

这个 @zrr1999 有经验,#61529 里也遇到过这个问题,报错看起来也是一样的(词法上的),应该是这里的 WARP_SIZE 与其他地方 define 的 WARP_SIZE 冲突,宏展开后 template <int WARP_SIZE> 变为 template <int 32> 导致词法分析时就出问题了,解决方案也很简单,要么改模板参数名字,要么改宏的名字

是这样,是在这个PR中修改 #64051

@co63oc co63oc closed this May 9, 2024
@co63oc co63oc deleted the fix_layer_norm2 branch May 12, 2024 06:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
contributor External developers
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants