Unused arguments in CMSIS-NN wrapper functions

Hi, we experimented with the CMSIS-NN demo introduced here: https://github.com/apache/tvm/pull/11013 Thanks a lot for enabling this!

It seems that parameters of your functions are unused in the case of the wrapped call to arm_convolve_wrapper_s8, namely filter_scale_ and input_scale.

Plus, according to the function signature they are int32 but they get a float array in the main function. I added an image of the main_0 function.

Could you explain the reason why these arguments exist? Thanks in advance!

@grant-arm @manupa-arm @ashutosh-arm @UlrikHjort @Khoi @MJKlaiberBosch @MJKlaiber

3 Likes

@Mousius for visibility

@grant-arm @Mousius Any updates on this? Sorry to bother you, but for us the CMSIS functionality is an awesome feature.

Hi @SebastianBoblestETAS,

I believe this PR (https://github.com/apache/tvm/pull/11431) by @ashutosh-arm addresses the issue. It seems to be failing in CI at the moment. Today is a holiday in the UK, so it’s likely to be Monday before it gets looked at again.

Hi @grant-arm, thanks a lot for the quick reply! Have a nice weekend!

1 Like

This https://github.com/apache/tvm/pull/11431 has been merged today.

Hi @SebastianBoblestETAS . this PR has been merged now.

2 Likes