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.

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

More flexible conv2d_NCHWc_int8 generic operator. #6714

Merged
merged 1 commit into from
Nov 8, 2020
Merged

More flexible conv2d_NCHWc_int8 generic operator. #6714

merged 1 commit into from
Nov 8, 2020

Conversation

cbalint13
Copy link
Contributor

@cbalint13 cbalint13 commented Oct 20, 2020

This small PR enable flexible parametrizations for conv2d_NCHWc_int8() int terms of int32_lanes and int8_elements .


Goal:

  • Allow conv2d_NCHWc_int8() to be called with any custom values of int32_lanes and int8_elements
  • Such flexibility (custom lane&elements) will be desired for future customizable ISA.

Changes:

  1. Make n_elems (default=4) optional parameter instead of static n_elems = 4 hardcode.
  2. Default fallback config now emit valid configuration for cases whenint32_lane < int8_num_elements too.
  3. All old behavior is kept, so actual Intel or ARM specific intrinsic will benefit the same default values as before.

@anijain2305 @zhiics @vinx13 @ZihengJiang please help me with the review,

Thank You !

@giuseros
Copy link
Contributor

Thanks for the explanation and for the pointers @cbalint13 ! This LGTM

@cbalint13
Copy link
Contributor Author

Thanks for the explanation and for the pointers @cbalint13 ! This LGTM

@giuseros , Thank you very much !

Copy link
Contributor

@giuseros giuseros left a comment

Choose a reason for hiding this comment

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

I didn't know I had the possibility to approve PRs without being a "reviewer". Apparently yes :)

@vinx13
Copy link
Member

vinx13 commented Nov 7, 2020

@cbalint13 could you try restarting the CI?

@cbalint13
Copy link
Contributor Author

@cbalint13 could you try restarting the CI?

@vinx13 , done, passed.

Thank You !

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants