[Test] Move cpp unittests to test directory#75632
[Test] Move cpp unittests to test directory#75632SigureMo merged 15 commits intoPaddlePaddle:developfrom
Conversation
Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com>
|
你的PR提交成功，感谢你对开源项目的贡献! |
There was a problem hiding this comment.
Pull Request Overview
This PR moves C++ unit tests from the paddle/fluid/framework/ir/ directory to the new test/cpp/legacy_ir/ directory, reorganizing the test structure to consolidate tests under the unified test directory.
- Migrates all legacy IR pass tests from source directory to dedicated test directory
- Creates new CMakeLists.txt files organizing tests by functionality (onednn, xpu, fusion_group)
- Updates main CMakeLists.txt to include the new legacy_ir subdirectory
Reviewed Changes
Copilot reviewed 8 out of 72 changed files in this pull request and generated 2 comments.
Show a summary per file
| File | Description |
|---|---|
| test/cpp/CMakeLists.txt | Adds subdirectory reference for legacy_ir tests |
| test/cpp/legacy_ir/CMakeLists.txt | Main CMakeLists organizing all legacy IR tests |
| test/cpp/legacy_ir/onednn/CMakeLists.txt | OneDNN-specific test configuration |
| test/cpp/legacy_ir/xpu/CMakeLists.txt | XPU-specific test configuration |
| test/cpp/legacy_ir/fusion_group/CMakeLists.txt | Fusion group test configuration |
| test/cpp/legacy_ir/op_compat_sensible_pass_test.cc | Reorders include statements |
| test/cpp/legacy_ir/generate_pass_test.cc | Reorders include statements |
| paddle/fluid/framework/ir/CMakeLists.txt | Removes all test definitions (290+ lines) |
Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
SigureMo
requested review from
XiaoguangHu01,
Xreki and
zhiqiu
as code owners
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## develop #75632 +/- ##
===========================================
Coverage ? 100.00%
===========================================
Files ? 2
Lines ? 9
Branches ? 0
===========================================
Hits ? 9
Misses ? 0
Partials ? 0 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
There was a problem hiding this comment.
Pull Request Overview
Copilot reviewed 13 out of 77 changed files in this pull request and generated 3 comments.
Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
--------- Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com> Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
PR Category
Environment Adaptation
PR Types
Improvements
Description
将
paddle/fluid/framework/ir/单测统一移到test/cpp/fluid/framework/ir/目录Co-authored with GitHub Copilot Agent with Claude Sonnet 4.5 model