On 8/22/24 1:46 PM, Patrick O'Neill wrote:
These cases are handled in the expander
(riscv-v.cc:expand_const_vector). We need the vector builder to detect
these cases so extract that out into a new riscv-v.h header file.
gcc/ChangeLog:
* config/riscv/riscv-v.cc (class rvv_builder): Move to riscv-v.h.
* config/riscv/riscv.cc (riscv_const_insns): Emit placeholder costs for
bool/stepped const vectors.
* config/riscv/riscv-v.h: New file.
OK. As you noted the pre-commit tester caught a dependency issue with
the missing return. I don't mind if you push with that fixed or if you
wait for the full series to get ACK'd. Either is fine with me.
jeff
