openhwgroup / corev-binutils-gdb

GNU General Public License v2.0
9 stars 26 forks source link

Updated hardware loop instruction encodings #70

Closed MaryBennett closed 1 year ago

MaryBennett commented 1 year ago

==Related Issues== Issues #63 & #64

==Commit==

Changed how uimml in cv.starti, cv.endi, and cv.setup was encoded to match the specification. Did the same for uimmS in cv.setupi.

Files Changed:

bfd:

gas/config:

gas/testsuite/gas/riscv:

ld/testsuite/ld-riscv-elf:

opcodes:

==Results==

===GAS=== Category Previous With commit Delta
Expected passes 1270 1270 -
Unexpected failures - - -
Unexpected successes - - -
Expected failures 23 23 -
Unresolved testcases - - -
Unsupported tests 9 9 -
===LD=== Category Previous With commit Delta
Expected passes 584 588 +4
Unexpected failures 18 14 -4
Unexpected successes - - -
Expected failures 18 18 -
Unresolved testcases - - -
Unsupported tests 219 219 -

The additional four passes in LD are due to:

jeremybennett commented 1 year ago

@MaryBennett - what are the remaining 14 unexpected failures?

MaryBennett commented 1 year ago

Two of the failures are caused by cv.bneimm and cv.beqimm. These are fixed in PR #72.

MaryBennett commented 1 year ago
Here are the new results: Category Previous With commit Delta
Expected passes 427 431 +4
Unexpected failures 6 2 -4
Unexpected successes - - -
Expected failures 16 16 -
Unresolved testcases 3 3 -
Untested testcases 20 20 -
Unsupported tests 200 200 -