[PATCH 2/2] RISC-V: Update testcases with new vector check.
Nelson Chu
nelson@rivosinc.com
Wed Jan 22 00:04:38 GMT 2025
On Tue, Jan 21, 2025 at 11:49 PM Jan Beulich <jbeulich@suse.com> wrote:
> On 21.01.2025 16:23, Jiawei wrote:
> > gas/ChangeLog:
> >
> > * testsuite/gas/riscv/dw-regnums.d: Add m in march.
> > * testsuite/gas/riscv/imply.d: Ditto.
> > * testsuite/gas/riscv/imply.s: Ditto.
> > * testsuite/gas/riscv/insn-na.d: Ditto.
> > * testsuite/gas/riscv/insn.d: Ditto.
> > * testsuite/gas/riscv/sifive-insns.s: Ditto.
> > * testsuite/gas/riscv/vector-insns-fail-vsew.d: Ditto.
> > * testsuite/gas/riscv/vector-insns-vmsgtvx.d: Ditto.
> > * testsuite/gas/riscv/vector-insns-zero-imm.d: Ditto.
> > * testsuite/gas/riscv/vector-insns.d: Ditto.
> > * testsuite/gas/riscv/zvfbfmin.d: Ditto.
> > * testsuite/gas/riscv/zvfbfwma.d: Ditto.
>
> Doesn't this change need to come first, to avoid breaking the testsuite
> intermediately?
>
I think only the imply testcase is needed, since previous patch v implies
m, so adding m into the architecture string is redundant. And it would be
better to update the imply testcase with the previous change.
Nelson
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://sourceware.org/pipermail/binutils/attachments/20250122/c9d6ad36/attachment-0001.htm>
More information about the Binutils
mailing list