[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