Re: [PATCH] RISC-V: Add configure check for Zaamo/Zalrsc assembler support

2024-06-17 Thread Patrick O'Neill
On 6/13/24 13:02, Jeff Law wrote: On 6/12/24 5:20 PM, Patrick O'Neill wrote: Binutils 2.42 and before don't support Zaamo/Zalrsc. Add a configure check to prevent emitting Zaamo/Zalrsc in the arch string when the assember does not support it. gcc/ChangeLog: * common/config/riscv/riscv-

Re: [PATCH] RISC-V: Add configure check for Zaamo/Zalrsc assembler support

2024-06-13 Thread Jeff Law
On 6/12/24 5:20 PM, Patrick O'Neill wrote: Binutils 2.42 and before don't support Zaamo/Zalrsc. Add a configure check to prevent emitting Zaamo/Zalrsc in the arch string when the assember does not support it. gcc/ChangeLog: * common/config/riscv/riscv-common.cc (riscv_subse

Re: [PATCH] RISC-V: Add configure check for Zaamo/Zalrsc assembler support

2024-06-12 Thread Sam James
Palmer Dabbelt writes: > On Wed, 12 Jun 2024 16:56:09 PDT (-0700), Patrick O'Neill wrote: >> >> On 6/12/24 16:49, Sam James wrote: >>> Palmer Dabbelt writes: >>> On Wed, 12 Jun 2024 16:20:26 PDT (-0700), Patrick O'Neill wrote: > Binutils 2.42 and before don't support Zaamo/Zalrsc. Add a

Re: [PATCH] RISC-V: Add configure check for Zaamo/Zalrsc assembler support

2024-06-12 Thread Palmer Dabbelt
On Wed, 12 Jun 2024 16:56:09 PDT (-0700), Patrick O'Neill wrote: On 6/12/24 16:49, Sam James wrote: Palmer Dabbelt writes: On Wed, 12 Jun 2024 16:20:26 PDT (-0700), Patrick O'Neill wrote: Binutils 2.42 and before don't support Zaamo/Zalrsc. Add a configure check to prevent emitting Zaamo/Za

Re: [PATCH] RISC-V: Add configure check for Zaamo/Zalrsc assembler support

2024-06-12 Thread Patrick O'Neill
On 6/12/24 16:49, Sam James wrote: Palmer Dabbelt writes: On Wed, 12 Jun 2024 16:20:26 PDT (-0700), Patrick O'Neill wrote: Binutils 2.42 and before don't support Zaamo/Zalrsc. Add a configure check to prevent emitting Zaamo/Zalrsc in the arch string when the assember does not support it. S

Re: [PATCH] RISC-V: Add configure check for Zaamo/Zalrsc assembler support

2024-06-12 Thread Sam James
Palmer Dabbelt writes: > On Wed, 12 Jun 2024 16:20:26 PDT (-0700), Patrick O'Neill wrote: >> Binutils 2.42 and before don't support Zaamo/Zalrsc. Add a configure >> check to prevent emitting Zaamo/Zalrsc in the arch string when the >> assember does not support it. > > Should we just rewrite these

Re: [PATCH] RISC-V: Add configure check for Zaamo/Zalrsc assembler support

2024-06-12 Thread Palmer Dabbelt
On Wed, 12 Jun 2024 16:20:26 PDT (-0700), Patrick O'Neill wrote: Binutils 2.42 and before don't support Zaamo/Zalrsc. Add a configure check to prevent emitting Zaamo/Zalrsc in the arch string when the assember does not support it. Should we just rewrite these to A when binutils doesn't support

[PATCH] RISC-V: Add configure check for Zaamo/Zalrsc assembler support

2024-06-12 Thread Patrick O'Neill
Binutils 2.42 and before don't support Zaamo/Zalrsc. Add a configure check to prevent emitting Zaamo/Zalrsc in the arch string when the assember does not support it. gcc/ChangeLog: * common/config/riscv/riscv-common.cc (riscv_subset_list::to_string): Skip zaamo/zalrsc when not