-
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
[PIR] move pir, hlir code generating from make stage to cmake stage #59129
Conversation
你的PR提交成功,感谢你对开源项目的贡献! |
endif() | ||
endforeach() | ||
|
||
add_custom_target( |
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.
可以先推动合入,后续完善
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 先推动合入,后续完善
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.
Great work!
…addlePaddle#59129) * initial modifying * fix error for copy_if_different * add add gitignore * modify paddle/cinn/hlir/dialect/operator/ir/CMakeLists.txt * modify gitignore * fix * polish * remove include_directories(${CMAKE_CURRENT_BINARY_DIR}) * revert delete include_directories(${CMAKE_CURRENT_BINARY_DIR})
PR types
Others
PR changes
Others
Description
主要工作
将
paddle/fluid/pir/dialect/CMakeLists.txt
的代码生成逻辑从 build 阶段放到 cmake 阶段,由 cmake 生成对应的pd_op.h
pd_op.cc
pd_op_vjp.cc
pd_api.h
pd_api.cc
static_op_function.h
static_op_function.cc
ops_api.cc
将
paddle/cinn/hlir/dialect/operator/ir/CMakeLists.txt
的代码生成逻辑从 build 阶段放到 cmake 阶段,由 cmake 生成对应的ops.parsed.yaml
cinn_op.cc
cinn_op.h
存在风险:
没有改变 include 的路径,使得增量编译存在新老两个自动生成的头文件时,一个存在于 source 路径下,一个存在于 build 路径下,若不清理 build/paddle/fluid/pir 和 build/paddle/cinn 目录,会错误的引用老的头文件(即 build/ 目录下的头文件)。例如:
进而产生变量未声明的编译错误