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

💡 Tip:

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

CLA Signature Pass

wangrui_, thanks for your pull request. All authors of the commits have signed the CLA. 👍

compile

流水线任务触发成功
任务链接 [1f314aa3b79047439460415cd13e9238][流水线指导]

[2026-06-18 14:23:21]    CI执行结束

流水线任务触发成功
任务链接 [7eaddc67a0a446a0b38ae6b91c050caa][流水线指导]

[2026-06-18 14:19:45]    CI执行结束

Now let me write the summary.

变更摘要

此 PR 主要修复了 reduce_log_sum_exp、reduce_max、reduce_min、reduce_prod 四个 ONNX 插件的实现。核心变更包括：为各算子补齐 noop_with_empty_axes 属性的存储/传递/设置流程；拆分 ONNX opset 8-12 与 13-18 的注册路径（PartitionedCall vs 算子名），并新增 opset 13+ 对应的 ParseParams*13 / ParseOpToGraph*13 函数；移除冗余的 GetInputTensorDimNum 辅助函数，改为在 opset 13+ 路径中统一通过 GetAttr("axes") 获取空轴张量。reduce_log_sum_exp 改动量最大，单独新增了 ParseParamsReduceLogSumExp13 等完整链条。

主要改动

• 分离 ONNX opset 8-12 与 13-18 注册路径: 将 reduce_max / reduce_min / reduce_prod / reduce_log_sum_exp 的 REGISTER_CUSTOM_OP 拆为两组："PartitionedCall" 覆盖 opset 8-12，新增 "ReduceMax" / "ReduceMin" / "ReduceProd" / "ReduceLogSumExp" 覆盖 opset 13-18，各自绑定对应版本的 ParseParams / ParseOpToGraph 函数。
• 补齐 noop_with_empty_axes 处理 (reduce_max/min/prod): 在 opset 8-12 的 ParseParamsReduce* 中存储 noop_with_empty_axes=0，在对应的 ParseOpToGraphReduce* 中读取并调用 set_attr_noop_with_empty_axes；opset 13+ 路径则在 ParseParamsReduce*13 中从 ONNX 节点属性读取真实值并在构建图时设置。
• 移除 GetInputTensorDimNum，改用空轴张量传递: 删除了三个文件中的 GetInputTensorDimNum 静态函数。opset 13+ 的 ParseParamsReduce*13 统一在 input_size==1 时存储一个空轴张量（dims={0}），由 ParseOpToGraphReduce*13 通过 GetAttr("axes") 获取，不再运行时查询输入维度计算全轴列表。
• reduce_log_sum_exp 新增 opset 13+ 完整支持: 新增 ParseParamsReduceLogSumExp13、匿名命名空间内的 ReduceLogSumExp13Prop 与 GetProperty、ParseOpToGraphReduceLogSumExp13，结构对齐其他三个算子，但未处理 noop_with_empty_axes 属性（存在不一致）。
• reduce_log_sum_exp: noop_with_empty_axes 检查从 else if 改为独立 if: ParseParamsReduceLogSumExp 中对该属性的检查不再受 keepdims 的 else if 约束，结构更清晰，无功能变化。

代码审查

All four files have been thoroughly reviewed. The reduce_max, reduce_min, and reduce_prod changes are consistent and correct — they add noop_with_empty_axes handling (matching their GE op definitions), remove the GetInputTensorDimNum/explicit-axis-computation pattern in favor of delegated empty-axes handling, and fix the opset 13 path where empty_axes == 1 would previously error out. The reduce_log_sum_exp changes correctly omit noop_with_empty_axes (its GE op lacks that attribute) while aligning with the same architecture.

审查总结

• P0: 0
• P1: 0
• P2: 0
• P3: 1（int num = v_axes.size() 窄化转换）

整体风险判断: 低风险。四个文件的改动逻辑一致、边界处理正确。唯一发现是一个低严重度的类型窄化问题，在实际场景中不会触发。

各文件审查结果:

• math/reduce_log_sum_exp/framework/reduce_log_sum_exp_onnx_plugin.cpp — 1 个 P3 问题
• math/reduce_max/framework/reduce_max_onnx_plugin.cpp — 无问题
• math/reduce_min/framework/reduce_min_onnx_plugin.cpp — 无问题
• math/reduce_prod/framework/reduce_prod_onnx_plugin.cpp — 无问题

💬 仅评论

/lgtm
/approve

/lgtm
/approve