cv.bset and cv.bclr builtin returns different length to rd

[MaryBennett]

For intructions cv.bset and cv.bclr, the destination register is a uint16_t but the builtin function returns a uint32_t. Is this correct?

[jeremybennett]

This is definitely wrong. The specification should be as follows:

uint32_t __builtin_riscv_cv_bitmanip_cv.bclr (uint32_t i, uint16_t range)

Case a) range is a constant

• result: rD
• i: rs1
• range[4:0]: Is2 (5-bit unsigned value)
• range[9:5]: Is3 (5-bit unsigned value)

or case b)

• result: rD
• i: rs1
• range: rs2

Generated assembler:

Case a)

        cv.bclr  rD,rs1,Is2,Is3

or case b)

        cv.bclrr  rD,rs1,rs2

uint32_t __builtin_riscv_cv_bitmanip_cv.bset (uint32_t i, uint16_t range)

Case a) range is a constant

• result: rD
• i: rs1
• range[4:0]: Is2 (5-bit unsigned value)
• range[9:5]: Is3 (5-bit unsigned value)

or case b)

• result: rD
• i: rs1
• range: rs2

Generated assembler:

Case a)

        cv.bset  rD,rs1,Is2,Is3

or case b)

        cv.bsetr  rD,rs1,rs2