core/blas/blasplan/BlasCalPlan.cpp 第 66 行,SetMaskTensor() 末尾:
SetMaskTensor()
toAclTensor(maskTensor, maskAclTensor); maskData = nullptr; // <- 该行 return ErrorType::ACL_SUCCESS;
maskData 是函数局部变量,置空后立即 return,该赋值无任何作用。hostData 所有权实际已转移给成员变量 maskTensor.hostData,FreeTensor() 中正确释放,故无功能问题,但这行死语句会误导读者以为置空有保护作用。
maskData
maskTensor.hostData
FreeTensor()
零功能影响,纯可读性问题。删除该行编译通过即无回归。
删除该行死语句;若担心后续维护误解,可在 toAclTensor 调用处加注释说明 hostData 所有权已转移。
toAclTensor
感谢您的反馈，当前@east_yang跟踪处理中。
Describe the current behavior / 问题描述
core/blas/blasplan/BlasCalPlan.cpp 第 66 行,
SetMaskTensor()末尾:toAclTensor(maskTensor, maskAclTensor); maskData = nullptr; // <- 该行 return ErrorType::ACL_SUCCESS;maskData是函数局部变量,置空后立即 return,该赋值无任何作用。hostData 所有权实际已转移给成员变量maskTensor.hostData,FreeTensor()中正确释放,故无功能问题,但这行死语句会误导读者以为置空有保护作用。Impact / 影响
零功能影响,纯可读性问题。删除该行编译通过即无回归。
Suggested fix / 修复建议
删除该行死语句;若担心后续维护误解,可在
toAclTensor调用处加注释说明 hostData 所有权已转移。Environment / 环境信息