Support feed op new ir #54840
Support feed op new ir #54840
Conversation
… support_fetch_in_new_ir
… support_feed_op_new_ir
|
你的PR提交成功，感谢你对开源项目的贡献! |
|
❌ The PR is not created using PR's template. You can refer to this Demo. |
| PADDLE_THROW(phi::errors::Unimplemented( | ||
| "only support dense tensor in vector type for now")); | ||
| } else if (result_type.isa<ir::VectorType>()) { | ||
| auto pos1 = result_type.dyn_cast<ir::VectorType>().data()[0]; |
There was a problem hiding this comment.
这里直接取了[0]，是说当result_type为vector时，内在元素至少有1个？
There was a problem hiding this comment.
这个问题 和下面的问题 是一起的，原来仅支持单个输出， 我升级了多输出，但是这块的逻辑没有适配对
| } | ||
|
|
||
| ir::Type t1 = ir::VectorType::get(ctx, op_output_types); | ||
| op_output_types.clear(); |
There was a problem hiding this comment.
这里为什么要clear op_output_types？ op_output_types的每个index对应的不是result_type所在的位置么，这里的clear是放在其中的一个分支里的。
对于：[Tensor0, [Tensor1], Tensor2]，会导致最后的 op_output_types 只是 [[Tensor1], Tensor2] ?
There was a problem hiding this comment.
这块确实是一个bug，仅在有一个 VectorType的情况是对的，其他的场景是有bug的，我看看怎么修复
PR types
Others
PR changes
Others
Description
支持feed op
Other
Pcard-67164