Skip to content

Conversation

@mklimenk
Copy link
Contributor

@mklimenk mklimenk commented Nov 3, 2025

Details:

Experimentally enable convolution with uint8 weights to request a oneDNN implementation to improve performance over a reference SIMD8 OpenCL implementation.

Tickets:

@mklimenk mklimenk requested review from a team as code owners November 3, 2025 08:42
@github-actions github-actions bot added the category: GPU OpenVINO GPU plugin label Nov 3, 2025
@sys-openvino-ci sys-openvino-ci added the ExternalIntelPR External contributor from Intel label Nov 3, 2025
one_of(out_dt, {data_types::i32, data_types::f16, data_types::f32, data_types::u8, data_types::i8});

if (!f16_conv && !u8s8_conv)
if (!f16_conv && !int8_conv)
Copy link
Contributor

Choose a reason for hiding this comment

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

could you add a unittest for this case?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Added in 54638e5

Copy link
Contributor

Choose a reason for hiding this comment

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

this int8_conv is little confusing, what about 8bit_conv or sth similar?

Copy link
Contributor

Choose a reason for hiding this comment

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

hmm it is good name

@mklimenk
Copy link
Contributor Author

mklimenk commented Nov 3, 2025

build_jenkins

Copy link
Contributor

@isanghao isanghao left a comment

Choose a reason for hiding this comment

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

lgtm

@isanghao isanghao enabled auto-merge November 4, 2025 05:53
@isanghao
Copy link
Contributor

isanghao commented Nov 4, 2025

This PR is related #32501

@isanghao
Copy link
Contributor

isanghao commented Nov 4, 2025

build_jenkins

@isanghao isanghao added this pull request to the merge queue Nov 4, 2025
Merged via the queue into openvinotoolkit:master with commit 7ebfc5c Nov 4, 2025
188 checks passed
IamRam3 pushed a commit to IamRam3/openvino that referenced this pull request Nov 6, 2025
…kit#32650)

### Details:
Experimentally enable convolution with uint8 weights to request a oneDNN
implementation to improve performance over a reference SIMD8 OpenCL
implementation.

### Tickets:
 - https://jira.devtools.intel.com/browse/CVS-175989
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

category: GPU OpenVINO GPU plugin ExternalIntelPR External contributor from Intel

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants