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

Add dimOp, tieProductEqualOp. access constraint_func in SymbolTable. Lowing DenseTensorType. #56615

Merged
merged 11 commits into from
Aug 28, 2023

Conversation

liuruyan
Copy link
Contributor

@liuruyan liuruyan commented Aug 24, 2023

PR types

Others

PR changes

Others

Description

  1. 新增 dimOp, tieProductEqualOp. 这两个Op为一期任务约束强依赖Op。
  2. SymbolTable 中接入存储 tieProductEqualOp,并更改lookup,insert方法为模板方法。
  3. 新增相应单测文件,关于SymbolTable接入tieProductEqualOp单测放在tie_product_equal中
  4. 下沉 DenseTensorType 到 builtin_type 中,独立依赖ddim取代依赖phi,pd_type 中使用 using 保持原功能兼容。

Other

Pcard-67164

@liuruyan liuruyan changed the title Add dimOp, tieProductEqualOp. access constraint_func in SymbolTable. Add dimOp, tieProductEqualOp. access constraint_func in SymbolTable. Lowing DenseTensorType. Aug 26, 2023
@@ -17,15 +17,122 @@
#include "paddle/ir/core/type.h"
#include "paddle/ir/core/type_base.h"
#include "paddle/ir/core/utils.h"
#include "paddle/phi/common/layout.h"
Copy link
Contributor

Choose a reason for hiding this comment

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

不建议在paddle/ir里面产生对phi的依赖。

Copy link
Contributor Author

Choose a reason for hiding this comment

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

layout.h只是头文件,没有cc文件。在预处理阶段会展开,不会引入phi依赖。关于ddim的处理则是在编译时独立编包,ir_core 独立依赖 ddim。

ir_library(ir_core SRCS ${IR_SRCS})
ir_library(ir_core SRCS ${IR_SRCS} DEPS ddim)

Copy link
Contributor Author

Choose a reason for hiding this comment

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

拆分独立编译 ddim 的 pr 如下:#56351

Copy link
Contributor

@Aurelius84 Aurelius84 left a comment

Choose a reason for hiding this comment

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

LGTM

Copy link
Contributor

@zhangbopd zhangbopd left a comment

Choose a reason for hiding this comment

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

LGTM

@zhangbopd zhangbopd merged commit 589588f into PaddlePaddle:develop Aug 28, 2023
lxd-cumt pushed a commit to lxd-cumt/Paddle that referenced this pull request Aug 28, 2023
…Lowing DenseTensorType. (PaddlePaddle#56615)

* add symbolicDimProduct & symbolicDimMgr without method shape_constraint related.

* add pd_type.cc to ir_shape CMakeLists.

* add dimOp, tieProductEqualOp. access constraint_func in SymbolTable.

* put DenseTensorType into builtin_type.
BeingGod pushed a commit to BeingGod/Paddle that referenced this pull request Sep 9, 2023
…Lowing DenseTensorType. (PaddlePaddle#56615)

* add symbolicDimProduct & symbolicDimMgr without method shape_constraint related.

* add pd_type.cc to ir_shape CMakeLists.

* add dimOp, tieProductEqualOp. access constraint_func in SymbolTable.

* put DenseTensorType into builtin_type.
@liuruyan liuruyan deleted the shapeOp branch December 2, 2024 07:42
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.

4 participants