You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@tvm.apache.org by "gromero (via GitHub)" <gi...@apache.org> on 2023/01/30 19:14:50 UTC

[GitHub] [tvm] gromero commented on issue #13856: [Bug] CMSIS-NN BYOC fails with Zephyr 3.2

gromero commented on issue #13856:
URL: https://github.com/apache/tvm/issues/13856#issuecomment-1409194895

   @mehrdadh Nice you found the solution :-) Basically with `-DARM_MATH_AUTOVECTORIZE` you're avoid the problematic code in 
   1) `arm_nn_mat_mult_nt_t_s8.c` and 2) `arm_softmax_s8.c`.
   
    For the issue 1) I think that's fine if gcc is really autovectorizing the C code now being built (instead of the inline asm that caused the "impossible constraint" error), but for 2) issue, it's an ICE -- internal compiler error , so I think it points to a serious but in the compiler and must be investigated.
   
   I'm curious if there is any reason for selecting this particular tag and cherry-picking this particular commit?


-- 
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.

To unsubscribe, e-mail: commits-unsubscribe@tvm.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org