Chenhu Wang
Chenhu Wang
Are there a JIRA ticket, could you please put the jira number in the description? And if there is model in the ticket, could you please test on how much...
Is it possible to add test to cover this kind of serialization/deserialization failure in https://github.com/openvinotoolkit/openvino/tree/master/src/tests/functional/inference_engine/serialization?
> In general LGTM. Would be nice if we can run internal perf validation before merge. Sure, internal perf validation is WIP, thanks
> @chenhu-wang do we have perf validation results? @dmitry-gorokhov , the result is updated in ticket. Thanks!
> @chenhu-wang Please rebase the branch @dmitry-gorokhov, I have rebased on latest master, there are some failure appeared for convert and split node on CI on Windows. WIP to analysis....
> @a-sidorova , Thanks a lot for your investigation and input. For rsp, rbp, rsi, rdi, it's lower 8bit part reg does not have the same index with 64 bit...
> > @chenhu-wang Please rebase the branch > > @dmitry-gorokhov, I have rebased on latest master, there are some failure appeared for convert and split node on CI on Windows....
> > > > @a-sidorova , Thanks a lot for your investigation and input. For rsp, rbp, rsi, rdi, it's lower 8bit part reg does not have the same index...
> As we discussed offline @chenhu-wang could you please cover case when `src_prc != dst_prc` as well? > > It blocks my [PR#12395](https://github.com/openvinotoolkit/openvino/pull/12395). `load_emitter` and `store_emitter` are often used in...
@dmitry-gorokhov , could you please final review? Thanks.