Revision bd62a3563de0004936f6506fdc5921210cb3bf14 authored by Zalman Stern on 08 June 2023, 01:18:19 UTC, committed by GitHub on 08 June 2023, 01:18:19 UTC
* Completely rework how RISC-V vector intrinsics are called to avoid
issues iwth single element vectors being confused with scalars and
other conversions that can happen via using call_intrin.

Allows using any size vector. Only downside is splitting large vectors
no longer happens, but RISC-V allows an LMUL of 8, meaning a vector of
up to 8 times the vector register size will compile so this is much
less of an issue. Splitting larger vectors can be added.

Should also allow fractionaly LMUL in all cases, but this is not
verified.

* Significant refactor/rewrite of RISC V vector intrinsics
support. Should handle many more cases and be well on the way to
handling arbitary vector widths within the LMUL range.

More tests added to simd_op_check_riscv .

Likely well setup to move SVE2 to a similar approach, perhaps without
the full genearilty on vector lengths. (I.e. they may need to be
quantized to vscale, or offer better performance in that case.)

* Formatting fixes.

* More formatting.

* Don't try to convert void types to match expected vector type.

* Backout comment change that is no longer relevant.

* Fix failure in camera_pipe app. (Code to make sure vector types match
was being presented with a scalar only mismatch. Changed it to ignore
scalar to scalar cases.)

Address review feedback.

* One more review comment.

* Comment fix.

---------

Co-authored-by: Steven Johnson <srj@google.com>
1 parent 67eaff3
History

back to top