On 10/27/2015 01:49 PM, David Malcolm wrote:
gcc/testsuite/ChangeLog:
        * g++.dg/plugin/plugin.exp (plugin_test_list): Add the unittests
        plugin.
        * g++.dg/plugin/unittests.C: New file.
        * gcc.dg/plugin/plugin.exp (plugin_test_list): Add the unittests
        plugin.
        * gcc.dg/plugin/unittests.c: New file.
        * lib/plugin-support.exp (dg-plugin-run-gengtype): New function.
        (plugin-get-options): Handle dg-plugin-run-gengtype.
        * lib/prune.exp (prune_gcc_output): Process any DejaGnu output
        from the test, and convert into results at this level, prefixing
        with the testcase.
        * unittests/unittests-plugin.c: New file.
I've noted the concerns WRT hooking into the pruning code. I won't repeat them.

There's a fair amount of if 0, #if 0 code in here that needs cleaning up.

And we've got the meta question, should these tests live somewhere different.

So I don't see anything else that would cause me to reject this -- but I'm going hold off on final approval pending discussion on where the tests will live as that may have an impact on this infrastructure.

jeff

Reply via email to