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]


Reply via email to