Skip to content

Introduce ov element type visitor#18189

Merged
ilyachur merged 17 commits intoopenvinotoolkit:masterfrom
praasz:feature/ov-element-type-for-each-visitor
Jul 6, 2023
Merged

Introduce ov element type visitor#18189
ilyachur merged 17 commits intoopenvinotoolkit:masterfrom
praasz:feature/ov-element-type-for-each-visitor

Conversation

@praasz
Copy link
Copy Markdown
Contributor

@praasz praasz commented Jun 22, 2023

Details:

  • Simplify evaluate implementation and code code maintenance by automatic code generation over switch case and macro
  • Applied visitor reduce openvinolib.so by around 27 KiB (Ubuntu 20.04, gcc 9.4, release build no LTO).

Tickets:

N/A

@praasz praasz requested review from a team as code owners June 22, 2023 11:53
@github-actions github-actions bot added category: Core OpenVINO Core (aka ngraph) category: CPU OpenVINO CPU plugin labels Jun 22, 2023
@praasz praasz requested a review from a team as a code owner June 23, 2023 12:24
@praasz praasz requested a review from t-jankowski June 23, 2023 13:20
@github-actions github-actions bot removed the category: CPU OpenVINO CPU plugin label Jun 26, 2023
@mlukasze
Copy link
Copy Markdown
Contributor

mlukasze commented Jul 6, 2023

@ilyachur & @mitruska could you review it, please?

Copy link
Copy Markdown
Contributor

@ilyachur ilyachur left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The idea LGTM

@ilyachur ilyachur merged commit c1f26d1 into openvinotoolkit:master Jul 6, 2023
@ilyachur ilyachur added this to the 2023.1 milestone Jul 6, 2023
@praasz praasz deleted the feature/ov-element-type-for-each-visitor branch August 4, 2023 13:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

category: Core OpenVINO Core (aka ngraph)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants