Skip to content

[GPU] Split fusion tests into separate files#9553

Merged
vladimir-paramuzov merged 1 commit intoopenvinotoolkit:masterfrom
vladimir-paramuzov:split_fusion_tests
Jan 11, 2022
Merged

[GPU] Split fusion tests into separate files#9553
vladimir-paramuzov merged 1 commit intoopenvinotoolkit:masterfrom
vladimir-paramuzov:split_fusion_tests

Conversation

@vladimir-paramuzov
Copy link
Copy Markdown

@vladimir-paramuzov vladimir-paramuzov commented Jan 10, 2022 •
edited
Loading

  • fusings_gpu_test.cpp file is split into multiple file based on primitive type
  • params and test classes/structures are put into anonymous namespace to avoid multiple definitions of same class in different translation units

@vladimir-paramuzov vladimir-paramuzov added the category: GPU OpenVINO GPU plugin label Jan 10, 2022
@vladimir-paramuzov vladimir-paramuzov added this to the 2022.1 milestone Jan 10, 2022
@vladimir-paramuzov vladimir-paramuzov requested review from a team as code owners January 10, 2022 17:17
lznamens
lznamens approved these changes Jan 11, 2022
View reviewed changes
Copy link
Copy Markdown
Contributor

@lznamens lznamens left a comment

Choose a reason for hiding this comment

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

Changes improve readability of the code, LGTM.

@vladimir-paramuzov vladimir-paramuzov merged commit 062523c into openvinotoolkit:master Jan 11, 2022
@vladimir-paramuzov vladimir-paramuzov deleted the split_fusion_tests branch January 11, 2022 09:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@yeonbok yeonbok yeonbok approved these changes

+1 more reviewer

@lznamens lznamens lznamens approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

category: GPU OpenVINO GPU plugin

Projects

None yet

Milestone

2022.1

Development

Successfully merging this pull request may close these issues.

3 participants

@vladimir-paramuzov @yeonbok @lznamens