-
Notifications
You must be signed in to change notification settings - Fork 5.9k
【Allocator】Add allocator visitor #76349
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
Conversation
|
你的PR提交成功,感谢你对开源项目的贡献! |
Codecov Report❌ Patch coverage is Additional details and impacted files@@ Coverage Diff @@
## develop #76349 +/- ##
==========================================
Coverage ? 91.93%
==========================================
Files ? 10
Lines ? 62
Branches ? 0
==========================================
Hits ? 57
Misses ? 5
Partials ? 0 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
wanghuancoder
left a comment
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
* add allocator visitor * fix ci * fix dcu * fix cuda * fix cuda * fix cuda * fix ci * fix ci * fix ci * add ut * fix ci * fix ci * fix ci * fix ci * fix dcu * fix dcu * fix conflict * fix conflict
PR Category
Performance Optimization
PR Types
New features
Description
本PR升级Allocator基础设施,新增Allocator Visitor,解决旧装饰器模式下,非顶层Allocator功能难以拓展及暴露的问题(想要暴露一个方法,只能再虚基类中加上方法,并在链路上所有Allocator中进行实现),具体改动如下:
注:Visitor日后可根据需求进行升级(升级可变模板参数)
Pcard-67164