Skip to content

r300: remove compiler tests

Pavel Ondračka requested to merge ondracka/mesa:r300_remove_compiler_tests into main

There are only three very simple tests but several hundreds of lines of corresponding infrastructure. The tests never helped me to catch a real issue and in fact I've only seen them fail when I changed the corresponding pass semantics and forgot to update the tests (or like when we enabled r300 in debian-testing and AddressSanitizer complained that there was not a single free in the tests).

We now have a real CI and both piglit and dEQP contain tests that cover the regalloc and presub testing here and a shader-db run will catch if we fail the one omod optimization the optimize test is testing. So I think its time we just get rid of the tests.

@gawin what do you think?

Merge request reports