search

riscv-non-isa / riscv-arch-test

https://jira.riscv.org/browse/RVG-141?src=confmacro
Apache License 2.0
516 stars 204 forks source link

Zfa tests missing fmvp.d.x #456

Open davidharrishmc opened 7 months ago

davidharrishmc commented 7 months ago

PR #330 is pending to add support for Zfa. The PR is missing fmvp.d.x, a RV32_Zfa instruction.

I suggest accepting the PR and adding fmvp.d.x later. This issue is to make sure the missing instruction is not forgotten.

Accodring to the PR #330 discussion (March 6, 2024), riscv-isac need enhancement to support fmv.p.d.

https://github.com/riscv-non-isa/riscv-arch-test/pull/330

There is a typo in the riscv-ctg PR (riscv_ctg/data/fd.yaml): s/fmvh.d.x/fmvp.d.x After fixing the typo the test generation fails, because riscv-isac can't handle that instruction

allenjbaum commented 2 months ago

ISAC PR#330 appears to be merged now. Can this issue be closed, or is something still needed?

davidharrishmc commented 2 months ago

When PRs 489 and 490 are merged in, I’ll be able to run the flow and test.

On Sep 9, 2024, at 3:41 PM, Allen Baum @.***> wrote:

ISAC PR#330 appears to be merged now. Can this issue be closed, or is something still needed?

— Reply to this email directly, view it on GitHub https://github.com/riscv-non-isa/riscv-arch-test/issues/456#issuecomment-2339277174, or unsubscribe https://github.com/notifications/unsubscribe-auth/AR4AA34CGIK47NEULONQM6TZVYPZXAVCNFSM6AAAAABGTBADHWVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDGMZZGI3TOMJXGQ. You are receiving this because you authored the thread.