Skip to content

Conversation

@xiaoguoguo626807
Copy link
Contributor

PR Category

Execute Infrastructure

PR Types

Improvements

Description

pcard-67164

拆解部分头文件,避免暴露pir 无关的头文件

@paddle-bot
Copy link

paddle-bot bot commented Feb 7, 2025

你的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.


constexpr int kEmptyVarIndex = 0;

struct EventInter {
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

new_executor_def 中原本定义了两类数据结构:一类是 Instruction 的成员,一类是 Interpreter 用到的成员,这个 new_executor_base_def.h 建议直接迁移到 instruction 目录下,改成 instruction_def.h 吧

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

done

Copy link
Contributor

@zhangbo9674 zhangbo9674 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@xiaoguoguo626807 xiaoguoguo626807 changed the title 【custom】depart Custom depend header 【custom】split Custom depend header Feb 10, 2025
Copy link
Contributor

@zyfncg zyfncg left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM for setup.py.in

@@ -0,0 +1,52 @@
// Copyright (c) 2021 PaddlePaddle Authors. All Rights Reserved.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

2021 -> 2025

@@ -0,0 +1,45 @@
// Copyright (c) 2023 PaddlePaddle Authors. All Rights Reserved.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

2023 -> 2025

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

这些文件是移动,不是新建,所以没改创建日期

Copy link
Contributor

@XiaoguangHu01 XiaoguangHu01 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@xiaoguoguo626807 xiaoguoguo626807 merged commit 8af53a3 into PaddlePaddle:develop Feb 11, 2025
32 checks passed
@xiaoguoguo626807 xiaoguoguo626807 deleted the custom_depend branch February 11, 2025 01:24
YqGe585 pushed a commit to YqGe585/Paddle that referenced this pull request May 7, 2025
* [custom] dependency for custom_engine

* add extention

* add .cc

* add declear

* [GCU] Open more information for instruction

* revert fix

* Revert "add extention"

This reverts commit 0ea393b.

* fix conflict

* recover pd_op_to_kernel_pass

* modify setup.py

* fix build

* fix build

* ercover deviceevent

* modify header name

* modify xpu build

* delete //

* rename new_executor_base_defs.h

* rename new_executor_base_defs.h

---------

Co-authored-by: EnflameGCU <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants