[PATCH v3 01/13] RISC-V: add dedicated vector arithmetic .insn forms

Jan Beulich jbeulich@suse.com
Mon Jul 20 05:59:50 GMT 2026


On 19.07.2026 12:33, Nelson Chu wrote:
> LGTM +1.

Thanks for taking a look.

>  A minor thing (this patch is already good enough to me
> without any change) is do you think we need to define extra MASK/MATCH
> for funct3 to represent bits 10-12 rather than use 0x7000 directly?
> So that might make it immediately clear to understand what these
> values represent.

Personally I'm really of the opposite opinion: The huge amount of MASK /
MATCH pairs is clutter. See how e.g. aarch64-tbl.h gets away (almost?)
completely without. And it's (imo) better readable that way.

Jan


More information about the Binutils mailing list