openvino
openvino copied to clipboard
openvinotoolkit
[Snippets] Added dynamism support to lowered passes
Details:
- Added dynamism support to lowered passes
- Brgemm related passes aren't affected (will be updated in the corresponding tasks to dynamic brgemm support)
Tickets:
- 123272
Prerequisites:
- https://github.com/openvinotoolkit/openvino/pull/21934
- https://github.com/openvinotoolkit/openvino/pull/21881
This PR will be closed in a week because of 2 weeks of no activity.
![github-actions[bot] avatar](https://avatars.githubusercontent.com/in/15368?v=4 "Published by a pub.dev verified publisher"){kind=small}
This PR will be closed in a week because of 2 weeks of no activity.
@IvanNovoselov @v-Golubev guys, thanks for the first review!
Let me invite you to the next review step 😃
I've added Validate pass to the final stage of control flow pipeline. It'd be great if you suggest additional checks to this pass!
Do you think we need a validation for this PR?
The PR contains specific conditions for dynamic case that cannot be validated at the moment. However, it affects the previous static pipeline. To be sure that the PR doesn't broke something, I've launched performance validation. The information (what is the commit, job) is attached in the ticket. I will ping you when the report is ready. Thanks!
@IvanNovoselov the validation has been passes. The results are attached to the ticket: no performance regressions.