Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Sns explicit tiles #13330

Closed
wants to merge 4 commits into from
Closed

Sns explicit tiles #13330

wants to merge 4 commits into from

Conversation

IvanNovoselov
Copy link
Contributor

@IvanNovoselov IvanNovoselov commented Oct 4, 2022 •
edited
Loading

Details:

  • Implement explicit planar tiles representation
  • Reformulate all presently implemented tile-related optimizations using this new representation

Tickets:

  • 92362

Yet to do:

  • Remove dynamism-related emitters
  • Refactor assign_registers to handle gp regs
  • Rename tile -> loop
  • Remove commented debug prints
  • Perform dlb validation

@IvanNovoselov IvanNovoselov self-assigned this Oct 4, 2022
@IvanNovoselov IvanNovoselov requested review from a team as code owners October 4, 2022 17:23
@dmitry-gorokhov dmitry-gorokhov added this to the 2022.3 milestone Oct 5, 2022
@IvanNovoselov IvanNovoselov force-pushed the sns_explicit_tiles branch 5 times, most recently from 2567c13 to 109872e Compare October 12, 2022 14:16
@IvanNovoselov IvanNovoselov force-pushed the sns_explicit_tiles branch 3 times, most recently from 9b6b8fc to 3b2137b Compare October 19, 2022 11:30
@IvanNovoselov
Copy link
Contributor Author

DLB validation is finished and revealed no issues. The results are in the ticket.

@IvanNovoselov IvanNovoselov force-pushed the sns_explicit_tiles branch 2 times, most recently from 540b4a2 to a411051 Compare November 7, 2022 10:24
@IvanNovoselov IvanNovoselov force-pushed the sns_explicit_tiles branch 2 times, most recently from 704804d to 3cecb89 Compare November 10, 2022 16:03
@dmitry-gorokhov dmitry-gorokhov removed this from the 2022.3 milestone Nov 16, 2022
@IvanNovoselov IvanNovoselov mentioned this pull request Nov 23, 2022
Merged
@IvanNovoselov
Copy link
Contributor Author

This PR will be merged as a part of a larger one PR#14327

@IvanNovoselov
Copy link
Contributor Author

Closing, since this code was merged on a separate branch

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@a-sidorova a-sidorova a-sidorova approved these changes

@dmitry-gorokhov dmitry-gorokhov Awaiting requested review from dmitry-gorokhov

Assignees

@dmitry-gorokhov dmitry-gorokhov

Labels
category: CPU OpenVINO CPU plugin do not merge do_not_merge
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@IvanNovoselov @a-sidorova @dmitry-gorokhov