已合并
fix: 修复 div/mod 类算子多项代码缺陷 #2111
zhang-song-rui创建于 4月7日
fix: 修复 div/mod 类算子多项代码缺陷 #2111
已合并
zhang-song-rui创建于 4月7日
zhang-song-rui
zhang-song-rui成员
4月7日

描述

  1. Addcdiv tiling 修复空指针先使用后判断:将 context->GetNodeName() 移到 null 检查之后
  2. aclnn_div HandleNotMixDataTypeDiv 修复参数传错:IsRealDivSupportNonContiguous 处理 other 时误传 self
  3. FloorMod/Mod tiling 修复 sizeof 使用错误:sizeof(input0Dtype) 改为 sizeof(float)
  4. Pow PowAiCore 添加 ADD_TO_LAUNCHER_LIST_AICORE 返回值检查
  5. Div l0op 删除 if-else 后不可达的 return divOut 并修改return写法

关联的Issue

https://gitcode.com/cann/ops-math/issues/1184

测试

文档更新

类型标签

likedislike
Pull Request已成功合入, 合并人@CANN-robot
(感谢 zhang-song-rui 的贡献)
zhang-song-ruizhang-song-rui成员
4月7日 创建了 pull request,commit becfc585
zhang-song-ruizhang-song-rui成员
4月7日 关联了issue:[Bug-Report|缺陷反馈]: Div/Mod 类算子多项代码缺陷修复
CANN-robot
CANN-robot成员
4月7日 评论:

Thanks for your pull-request.
The full list of commands accepted by me can be found at here
You can get sig-info at here


PR Approval Progress

Congratulations! All modules have met the lgtm and approve requirements.

Module Approval Details

module lgtm status approve status
math/addcdiv zhangzijie, songkai111 (2/2) zhangzijie (1/1)
math/div zhangzijie, songkai111 (2/2) zhangzijie (1/1)
math/floor_mod zhangzijie, songkai111 (2/2) zhangzijie (1/1)
math/mod zhangzijie, songkai111 (2/2) zhangzijie (1/1)
math/pow songkai111, zhangzijie (2/2) zhangzijie (1/1)

💡 Tip:

  • Committer can comment /approve or /lgtm
  • Commenting /approve implies both code review (lgtm) and intent to merge (approve)

CLA Signature Pass

zhang-song-rui, thanks for your pull request. All authors of the commits have signed the CLA. 👍

likedislike
CANN-robotCANN-robot成员
4月7日 添加了label:cann-cla/yes
CANN-robotCANN-robot成员
4月7日 将FelixTang7,zhou-qilong,wangrui_,rxtfeng,loov1,zhangzijie,songkai111,zhangyuxiang0119,llimwang,gubaocheng设为评审人
此处折叠了100条消息 查看更多
CANN-robot
CANN-robot成员
4月17日 评论:

The MR can not be merged, because of CodeReview discussion not resolved

If you want to solve this problem, you can click here to do it in the FAQs.

likedislike
Zzhangzijie成员
4月17日 解决了最后一个问题
CANN-robot
CANN-robot成员
4月17日 评论:

Review Guide

This pull-request passes review.
Committers who wrote a comment of /approve are: zhangzijie.
Reviewers who wrote a comment of /lgtm are: zhangzijie, songkai111.

likedislike
CANN-robotCANN-robot成员
4月17日 关闭了关联的issue
CANN-robotCANN-robot成员
4月17日 合入了pull request