cbalint13 opened a new pull request #6714:
URL: https://github.com/apache/incubator-tvm/pull/6714
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 (will publish soon such arch contrib).
**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
when```int32_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 !
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
[email protected]