| 【fix】: 修复 ArgListManager 单例实例分裂导致 TestGlobalOccupy 测试失败
Co-authored-by: zhang_shengjie<804425610@qq.com>
# message auto-generated for no-merge-commit merge:
!628 merge fix_ut_instant into develop
【fix】: 修复 ArgListManager 单例实例分裂导致 TestGlobalOccupy 测试失败
Created-by: zhang_shengjie
Commit-by: zhang_shengjie
Merged-by: cann-robot
Description:
# Pull Request
## 描述
### 一、主要解决的问题
#### 1.1 ArgListManager 单例实例分裂导致测试失败
**问题现象**：
- TestBufOccupyExpr.TestGlobalOccupy 测试在 aarch64 环境上失败
- 测试期望 GetTotalGlobalOccup 返回非 SUCCESS，但实际返回 SUCCESS
- 同一测试在 x86_64 环境上通过（假阳性）
**根因分析**：
- ArgListManager::GetInstance() 在头文件中 inline 定义，使用 Meyer's Singleton 模式
- att_ut 的 CMakeLists.txt 使用 -Wl,-Bsymbolic 链接选项
- -Bsymbolic 使可执行文件优先使用自己的符号，导致 att_ut 和 libatt.so 各自持有独立的单例实例
- x86_64 链接器行为导致实例分裂（假阳性通过），aarch64 链接器正确合并 weak 符号（暴露真实问题）
**影响范围**：
- 所有依赖 ArgListManager::GetInstance() 的代码可能存在单例分裂隐患
- 测试用例期望值本身错误（对有效 Workspace 应返回 SUCCESS）
#### 1.2 删除未使用的 AscirCompileAndLaunch 接口
**问题描述**：
- codegen_tiling.cpp 中存在大量未使用的 AscirCompileAndLaunch 相关函数
- 代码冗余，增加维护成本
**影响范围**：
- codegen_tiling.cpp 和 codegen_tiling.h
- 约 170 行代码删除
### 二、修改方案
#### 2.1 修复 ArgListManager 单例实例分裂
**方案**：将 GetInstance() 实现从头文件移到 .cpp 文件
**修改前** (arg_list_manager.h)：
```cpp
class ArgListManager {
public:
static ArgListManager &GetInstance() {
static ArgListManager arg_list_mgr;
return arg_list_mgr;
}
};
```
**修改后**：
- arg_list_manager.h：仅声明 static ArgListManager &GetInstance();
- arg_list_manager.cpp：实现 GetInstance() 函数
**原理**：
- 将 static 局部变量从 weak 符号（inline 函数）变为强符号（.cpp 文件）
- 确保全局只有一个单例实例，无论链接器如何处理 -Bsymbolic
#### 2.2 修正测试期望值
**修改** (test_buf_occupy_expr.cpp)：
```cpp
// 修改前
EXPECT_NE(buf_evaluator->GetTotalGlobalOccup(global_occup_expr), af::SUCCESS);
// 修改后
EXPECT_EQ(buf_evaluator->GetTotalGlobalOccup(global_occup_expr), af::SUCCESS);
```
**理由**：addResOut 是 Workspace 类型节点，有有效的 tensor 表达式，GetTotalGlobalOccup 应该返回 SUCCESS。
#### 2.3 删除 AscirCompileAndLaunch 接口
**删除的函数**：
- GenAutofuseLaunchDeclare()
- GenAscirCompileAndLaunchHead()
- GenAclInit()
- GenAscirTilingFunc()
- GenAscirMallocWorkspaceFunc()
- GenAscirLaunchFunc()
- TilingLib::GenAscirTilingAndLaunchFunc()
**删除的代码量**：约 170 行
## 变更类型
- [x] 🐛 Bug 修复
- [ ] ✨ 新功能
- [ ] 💄 代码风格更新
- [ ] ♻️ 重构
- [x] 📦 构建过程或辅助工具的变动
- [ ] 📝 文档内容更新
## 关联的Issue
Closes #82
## 如何测试
### 一、测试用例说明
#### 1.1 单元测试
**测试目标**：TestBufOccupyExpr.TestGlobalOccupy
**测试步骤**：
```bash
cmake --build build --target att_ut -j 8
./build/autofuse/tests/ut/att/att_ut --gtest_filter=TestBufOccupyExpr.TestGlobalOccupy
./build/autofuse/tests/ut/att/att_ut
```
**预期结果**：
- TestGlobalOccupy 测试通过
- 所有 931 个 att UT 测试通过
**实际验证结果**：
- ✅ 本地 x86_64 环境：931/931 通过
- ✅ 远端 CI 流水线：编译成功
#### 1.2 系统测试
**测试目标**：验证删除 AscirCompileAndLaunch 接口不影响现有功能
**测试步骤**：
```bash
bash scripts/test/run_autofuse_test.sh -s -m codegen -j 8
```
**预期结果**：所有 codegen ST 测试通过
## 核对清单
- [x] 我的代码遵循了项目的代码风格
- [x] 我已对代码进行了自测
- [x] 我已更新了相关的文档
- [x] 我在标题中使用了合适的类型标签
- [x] 我已经详细阅读了贡献指南（CONTRIBUTING.md）
## 其他信息
### 验证方法
1. **本地验证**：编译 att_ut 并运行所有测试，确认 TestGlobalOccupy 测试通过
2. **远端 CI 验证**：流水线 #696635：✅ 成功（23分钟）
### 注意事项
1. **单例模式最佳实践**：
- 避免在头文件中 inline 定义包含 static 局部变量的函数
- 如需确保全局唯一实例，将实现放在 .cpp 文件中
2. **-Bsymbolic 链接选项影响**：
- 该选项使可执行文件优先使用自己的符号
- 可能导致 weak 符号（如 inline 函数的 static 变量）实例分裂
- 在跨架构（x86_64 vs aarch64）时行为可能不同
3. **测试期望值审查**：
- 定期检查测试用例的期望值是否合理
- 避免因环境差异导致的假阳性通过
### 提交记录
| Commit | 描述 | 修改文件数 |
|--------|------|-----------|
| ecd463c | fix: 修复 ArgListManager 单例实例分裂导致 TestGlobalOccupy 测试失败 | 3 |
| 985b879 | 【fix】: 删除AscirCompileAndLaunch接口 | 2 |
### 修改文件清单
| 文件路径 | 修改类型 | 说明 |
|---------|---------|------|
| autofuse/att/gen_model_info/expr_gen/arg_list_manager.h | 修改 | 移除 GetInstance() inline 实现 |
| autofuse/att/gen_model_info/expr_gen/arg_list_manager.cpp | 修改 | 添加 GetInstance() 实现 |
| autofuse/tests/ut/att/testcase/gen_model_info/expr_gen/test_buf_occupy_expr.cpp | 修改 | 修正测试期望值 |
| autofuse/codegen/codegen_tiling.cpp | 修改 | 删除 AscirCompileAndLaunch 相关函数 |
| autofuse/codegen/codegen_tiling.h | 修改 | 删除 GenAscirTilingAndLaunchFunc 声明 |
See merge request: cann/graph-autofusion!628 | 14 小时前 |
