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

[PIR] remove add_custom_target in pir codegen of CMakeLists.txt #59222

Merged
merged 12 commits into from
Nov 24, 2023

Conversation

MarioLulab
Copy link
Contributor

@MarioLulab MarioLulab commented Nov 21, 2023

PR types

Others

PR changes

Others

Description

#59129 里将 pir 相关的代码生成从 make 编译阶段移动至了 cmake 构建阶段,
因此此时在 paddle/fluid/pir/dialect/CMakeLists.txt 中没有了 add_custom_command 相关命令。在没有显式指明 target 和没有其他 target 依赖该 target 时,命令不会执行。所以除了 ops_api_gen,将其他的 add_custom_target 从 paddle/fluid/pir/dialect/CMakeLists.txt 中删除

MarioLulab and others added 12 commits November 19, 2023 10:27
paddle/cinn/hlir/dialect/operator/ir/CMakeLists.txt
include_directories(${CMAKE_CURRENT_BINARY_DIR})
delete include_directories(${CMAKE_CURRENT_BINARY_DIR})
@paddle-bot paddle-bot bot added the contributor External developers label Nov 22, 2023
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

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

@Aurelius84 Aurelius84 merged commit 2004674 into PaddlePaddle:develop Nov 24, 2023
28 checks passed
SecretXV pushed a commit to SecretXV/Paddle that referenced this pull request Nov 28, 2023
…lePaddle#59222)

* 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})

* fix

* revert ops_api_gen
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
contributor External developers
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants