[testsuite] Fix plugin testsuite, remove uses of TODO_dump_func (PR testsuite/52948)
Rainer Orth
ro@CeBiTec.Uni-Bielefeld.DE
Mon Apr 16 10:24:00 GMT 2012
As reported in PR testsuite/52948, several plugin testcases were failing
since the removal of TODO_dump_func:
UNRESOLVED: selfassign.c compilation, -I. -I/vol/gcc/src/hg/trunk/local/gcc/testsuite -I/vol/gcc/src/hg/trunk/local/gcc/testsuite/../../gcc -I/var/gcc/regression/trunk/11-gcc/build/gcc/testsuite/g++/../../../gcc -I/vol/gcc/src/hg/trunk/local/gcc/testsuite/../../include -I/vol/gcc/src/hg/trunk/local/gcc/testsuite/../../libcpp/include -I/vol/gcc/include -I/vol/gcc/include -I/vol/gcc/include -I/var/gcc/regression/trunk/11-gcc/build/gcc/testsuite/g++/../../../intl -O -DIN_GCC -fPIC -shared
I have no idea why this has been missed when testing the removal patch,
because the failures are very prominent in mail-report.log (maybe
because the incorrectly show up as UNRESOLVED instead of FAIL).
This patch fixes this by removing the uses of TODO_dump_func from the
affected plugins.
At the same time, I'm finally fixing some issues with the plugin tests
that have annoyed me for quite some time:
* The tests should log the full (relative) path to the testcase, not
only the basenames.
* Using unresolved to report failures to compile the plugins is wrong:
this should be fail instead, and the pass case needs to be logged as
well.
* There's no point in logging the options used to compile the plugin:
they are completely internal to plugin-support.exp and not in any way
influenced by the testsuite drivers.
There's one issue I didn't fix due to the large number of (bad)
precedents in the testsuite:
* If a plugin fails to compile, the compilations using it should become
unresolved instead of just returning early.
Bootstrapped without regressions on i386-pc-solaris2.11, installed on
mainline.
Rainer
2012-04-14 Rainer Orth <ro@CeBiTec.Uni-Bielefeld.DE>
gcc/testsuite:
* lib/plugin-support.exp (plugin-test-execute): Properly determine
testcase name.
Use fail, pass instead of unresolved.
Don't log $optstr.
PR testsuite/52948
* g++.dg/plugin/dumb_plugin.c (pass_dumb_plugin_example): Remove
TODO_dump_func.
* g++.dg/plugin/selfassign.c (pass_warn_self_assign): Likewise.
* gcc.dg/plugin/one_time_plugin.c (one_pass): Likewise.
* gcc.dg/plugin/selfassign.c (pass_warn_self_assign): Likewise.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: testsuite-plugin.patch
Type: text/x-patch
Size: 3659 bytes
Desc: not available
URL: <http://gcc.gnu.org/pipermail/gcc-patches/attachments/20120416/59658f5f/attachment.bin>
-------------- next part --------------
--
-----------------------------------------------------------------------------
Rainer Orth, Center for Biotechnology, Bielefeld University
More information about the Gcc-patches
mailing list