[Libre-soc-isa] [Bug 686] create Power ISA test API

bugzilla-daemon at libre-soc.org bugzilla-daemon at libre-soc.org
Fri Sep 17 16:12:35 BST 2021


https://bugs.libre-soc.org/show_bug.cgi?id=686

--- Comment #51 from Luke Kenneth Casson Leighton <lkcl at lkcl.net> ---
https://libre-soc.org/irclog/%23libre-soc.2021-09-17.log.html#t2021-09-17T15:19:31

> kylel	so basically add an optional expected for add_case and then in check_regs choose the proper dict?

basically, yes.  the minor complication (not really) is that add_case
stores everything in a class, actually a list of class instances,
which the run_tst() picks up, enumerates, and runs as subTests
on just the one Simulator/HDL-runner

(we did this because it's so damn expensive to set up)

thus the ExpectedState() will need to be added as an optional
argument to test_data, in soc simple/test/test_runner.py

that will be... TestAccumulatorBase and TestCase.
https://git.libre-soc.org/?p=openpower-isa.git;a=blob;f=src/openpower/test/common.py;h=2bc0c1fab494720f388b24e2be598883381c535c;hb=HEAD#l117

* TestCase will need a new (optional) expected argument

* TestAccumulatorBase.add_case will need the same

* soc's simple/test/test_runner.py will need to *use* that

BUT...

BUTBUTBUT...

note that just like back in test_caller_shiftrot.py, those Expected
Results are **NOT** - cannot - be compared in the *MIDDLE* of the
test run.

they can only be called *AT THE END*.

i will update the source code with the exact point where that can
be done, with a TODO comment

-- 
You are receiving this mail because:
You are on the CC list for the bug.


More information about the Libre-SOC-ISA mailing list