CARVIEW |
Navigation Menu
-
Notifications
You must be signed in to change notification settings - Fork 5.8k
【custom】split device file #71085
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
【custom】split device file #71085
Conversation
你的PR提交成功,感谢你对开源项目的贡献! |
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
@@ -1174,20 +1174,20 @@ headers = ( | |||
list(find_files('token.h','@PADDLE_SOURCE_DIR@/paddle/pir/src/core/parser/'))+ | |||
#pir ops and dependency | |||
list(find_files('pd_op.h','@PADDLE_BINARY_DIR@/paddle/fluid/pir/dialect/operator/ir/'))+ | |||
list(find_files('utils_defs.h','@PADDLE_SOURCE_DIR@/paddle/fluid/ir_adaptor/translator/'))+ | |||
list(find_files('pd_op_sig.h','@PADDLE_SOURCE_DIR@/paddle/fluid/ir_adaptor/translator/'))+ |
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.
setup.py 看下是否需要同步修改
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.
setup.py 下个PR 修改
#pragma once | ||
#include <memory> | ||
|
||
#include "glog/logging.h" |
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.
头文件里不能带glog这样的三方库头文件,外部编译的时候可能会引起链接错误
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.
该头文件中实现和定义在一起,需要这个glog
#include "paddle/phi/backends/context_pool.h" | ||
#include "paddle/phi/backends/cpu/cpu_context.h" | ||
#include "paddle/phi/backends/custom/custom_context.h" | ||
#include "paddle/phi/backends/device_ext.h" | ||
#include "paddle/phi/backends/gpu/gpu_decls.h" | ||
#include "paddle/phi/common/place.h" | ||
#include "paddle/phi/core/device_context.h" | ||
#include "paddle/phi/core/enforce.h" |
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.
没有用到的头文件是不可以删一删
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.
单独提PR整理下
|
||
#include "glog/logging.h" | ||
#include "paddle/phi/core/platform/device_type.h" | ||
#include "paddle/utils/test_macros.h" |
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.
test_macros.h是否有用到?
* split device file * fix path
PR Category
Execute Infrastructure
PR Types
Improvements
Description
pcard-67164
拆分device_context.h device_event.h


修改set_up.py错误的路径