Re: [PATCH] RISC-V: Fix some generic-vector-ooo pipeline description issues

On 7/28/25 3:23 AM, Robin Dapp wrote: -;; Vector crypto, assumed to be a generic operation for now. -(define_insn_reservation "vec_crypto" 4 +;; Vector population count +(define_insn_reservation "vec_pop" 4    (and (eq_attr "tune" "generic_ooo,generic") -   (eq_attr "type" "crypto,vclz,vct

Re: [PATCH] RISC-V: Fix some generic-vector-ooo pipeline description issues

Hi Robin, > > -;; Vector crypto, assumed to be a generic operation for now. > > -(define_insn_reservation "vec_crypto" 4 > > +;; Vector population count > > +(define_insn_reservation "vec_pop" 4 > >(and (eq_attr "tune" "generic_ooo,generic") > > - (eq_attr "type" "crypto,vclz,vctz,vcpop"

Re: [PATCH] RISC-V: Fix some generic-vector-ooo pipeline description issues

-;; Vector crypto, assumed to be a generic operation for now. -(define_insn_reservation "vec_crypto" 4 +;; Vector population count +(define_insn_reservation "vec_pop" 4 (and (eq_attr "tune" "generic_ooo,generic") - (eq_attr "type" "crypto,vclz,vctz,vcpop")) + (eq_attr "type" "vcpop"

[PATCH] RISC-V: Fix some generic-vector-ooo pipeline description issues

The crypto type covers scalar (not vector) crypto instructions, and vclz and vctz appear in two places. gcc/ChangeLog: * config/riscv/generic-vector-ooo.md (vec_crypto): Remove crypto, vclz, vctz from insn reservation, rename to vec_pop. --- gcc/config/riscv/generic-vector-ooo.md