On 10/29/26 12:04 AM, Philippe Mathieu-Daudé wrote:
On 28/10/26 21:07, Pierrick Bouvier wrote:
On 10/28/2026 11:21 AM, Philippe Mathieu-Daudé wrote:
I'm glad we have many MIPS tests but they are in too
many files, taking time to get compiled. In particular
on Aarch64 host via x86_64 docker image... Group the
test in order to reduce CI build time.
Philippe Mathieu-Daudé (4):
tests/tcg/mips: Clear DSPControl at start of DSP tests
tests/tcg/mips: Group DSP tests
tests/tcg/mips: Group Release 6 tests
tests/tcg/mips: Group MSA tests
That's a good idea in itself, but this would need additional work to
have a clear report per test. We can't trade usability for faster
compilation.
Right now, a test will stop on the first assert() met, without going
further.
Ideally, we can keep the list of tests in a static table, and perform
the DSPControl clear before each, so patch 1 can be removed. Also,
assert could be replaced with a proper return and print information per
test.
IIUC the requested changes are only for the DSP tests, as the other
groups (Rel6 and MSA) do not stop on the first assert() hit.
Yes. Other patches still need to add some reporting on which test failed
(a print with OK/FAIL is enough) instead of just a global return code.
I'll rework patch 2 (indeed 1 is then unnecessary) and respin, thanks.
