Message ID | 20240327195403.29732-2-palmer@rivosinc.com |
---|---|
State | New |
Headers | show |
Series | RISC-V: Add vxsat as a register | expand |
LGTM, and committed to trunk :) On Thu, Mar 28, 2024 at 5:37 AM Palmer Dabbelt <palmer@rivosinc.com> wrote: > > We aren't doing anything with vxsat right now, but I'd like to add it as > an accepted register to the clobber list. If we get this into GCC-14 > then we'll avoid some preprocessor-based twiddling if we ever start > using vxsat in the future. > > gcc/ChangeLog: > > * config/riscv/riscv.h (REGISTER_NAMES): Add vxsat. > --- > IIUC we aren't using these N/A regnos for anything, they're just there to pad > out the types. So I think this is safe, but Juzhe would likely know best here. > > See > https://inbox.sourceware.org/libc-alpha/20240327193601.28903-2-palmer@rivosinc.com/ > a use of this. > --- > gcc/config/riscv/riscv.h | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/gcc/config/riscv/riscv.h b/gcc/config/riscv/riscv.h > index da089a03e9d..d5779512994 100644 > --- a/gcc/config/riscv/riscv.h > +++ b/gcc/config/riscv/riscv.h > @@ -933,7 +933,7 @@ extern enum riscv_cc get_riscv_cc (const rtx use); > "fs0", "fs1", "fa0", "fa1", "fa2", "fa3", "fa4", "fa5", \ > "fa6", "fa7", "fs2", "fs3", "fs4", "fs5", "fs6", "fs7", \ > "fs8", "fs9", "fs10","fs11","ft8", "ft9", "ft10","ft11", \ > - "arg", "frame", "vl", "vtype", "vxrm", "frm", "N/A", "N/A", \ > + "arg", "frame", "vl", "vtype", "vxrm", "frm", "vxsat", "N/A", \ > "N/A", "N/A", "N/A", "N/A", "N/A", "N/A", "N/A", "N/A", \ > "N/A", "N/A", "N/A", "N/A", "N/A", "N/A", "N/A", "N/A", \ > "N/A", "N/A", "N/A", "N/A", "N/A", "N/A", "N/A", "N/A", \ > -- > 2.44.0 >
diff --git a/gcc/config/riscv/riscv.h b/gcc/config/riscv/riscv.h index da089a03e9d..d5779512994 100644 --- a/gcc/config/riscv/riscv.h +++ b/gcc/config/riscv/riscv.h @@ -933,7 +933,7 @@ extern enum riscv_cc get_riscv_cc (const rtx use); "fs0", "fs1", "fa0", "fa1", "fa2", "fa3", "fa4", "fa5", \ "fa6", "fa7", "fs2", "fs3", "fs4", "fs5", "fs6", "fs7", \ "fs8", "fs9", "fs10","fs11","ft8", "ft9", "ft10","ft11", \ - "arg", "frame", "vl", "vtype", "vxrm", "frm", "N/A", "N/A", \ + "arg", "frame", "vl", "vtype", "vxrm", "frm", "vxsat", "N/A", \ "N/A", "N/A", "N/A", "N/A", "N/A", "N/A", "N/A", "N/A", \ "N/A", "N/A", "N/A", "N/A", "N/A", "N/A", "N/A", "N/A", \ "N/A", "N/A", "N/A", "N/A", "N/A", "N/A", "N/A", "N/A", \