On 1/17/25 2:30 AM, Jin Ma wrote:
gcc/testsuite/ChangeLog:

        * gcc.target/riscv/rvv/xtheadvector/pr114194.c: Adjust correctly.
        * gcc.target/riscv/rvv/xtheadvector/prefix.c: Likewise.
        * gcc.target/riscv/rvv/xtheadvector/vlb-vsb.c: Likewise.
        * gcc.target/riscv/rvv/xtheadvector/vlbu-vsb.c: Likewise.
        * gcc.target/riscv/rvv/xtheadvector/vlh-vsh.c: Likewise.
        * gcc.target/riscv/rvv/xtheadvector/vlhu-vsh.c: Likewise.
        * gcc.target/riscv/rvv/xtheadvector/vlw-vsw.c: Likewise.
        * gcc.target/riscv/rvv/xtheadvector/vlwu-vsw.c: Likewise.
        * gcc.target/riscv/rvv/xtheadvector/xtheadvector.exp: New test.
So the biggest question I have is why this patch adds xtheadvector.exp rather than just adding another directory to the rvv.exp driver.


While there's nothing inherently wrong with adding xtheadvector.exp, it seems better from a long term maintenance standpoint to rely on the rvv.exp driver if we can. By doing so the behavior of tests in the xtheadvector directory would mirror those in the other directories under rvv.

Jeff

Reply via email to