selftest: Move C-specific tests to c_family
When trying to make use of the selftest framework over on the rust frontend, we ran into issues where rust1 was expected to produce errors containing C-like type names such as `int`. I had gotten in contact with David Malcolm on the gcc mailing list [1], who advised moving some test functions to a better location. The offending functions have also been renamed in order to better fit the C family of tests, and are thus not called when performing general selftests anymore. Kindly, [1]: https://gcc.gnu.org/pipermail/gcc/2021-November/237703.html 2022-02-16 Arthur Cohen <arthur.cohen@embecosm.com> gcc/c-family/ * c-common.cc (c_family_tests): Call the new tests. * c-common.h (c_diagnostic_tests): Declare. (c_opt_problem_cc_tests): Likewise. gcc/ * diagnostic.cc (diagnostic_cc_tests): Rename to... (c_diagnostic_cc_tests): ...this. * opt-problem.cc (opt_problem_cc_tests): Rename to... (c_opt_problem_cc_tests): ...this. * selftest-run-tests.cc (selftest::run_tests): No longer run opt_problem_cc_tests or diagnostic_cc_tests. * selftest.h (diagnostic_cc_tests): Remove declaration. (opt_problem_cc_tests): Likewise.
This commit is contained in:
parent
bc84b61b00
commit
981524428b
@ -9120,6 +9120,8 @@ c_family_tests (void)
|
||||
c_indentation_cc_tests ();
|
||||
c_pretty_print_cc_tests ();
|
||||
c_spellcheck_cc_tests ();
|
||||
c_diagnostic_cc_tests ();
|
||||
c_opt_problem_cc_tests ();
|
||||
}
|
||||
|
||||
} // namespace selftest
|
||||
|
@ -1512,8 +1512,10 @@ extern tree braced_lists_to_strings (tree, tree);
|
||||
namespace selftest {
|
||||
/* Declarations for specific families of tests within c-family,
|
||||
by source file, in alphabetical order. */
|
||||
extern void c_diagnostic_cc_tests (void);
|
||||
extern void c_format_cc_tests (void);
|
||||
extern void c_indentation_cc_tests (void);
|
||||
extern void c_opt_problem_cc_tests (void);
|
||||
extern void c_pretty_print_cc_tests (void);
|
||||
extern void c_spellcheck_cc_tests (void);
|
||||
|
||||
|
@ -2472,7 +2472,7 @@ test_num_digits ()
|
||||
/* Run all of the selftests within this file. */
|
||||
|
||||
void
|
||||
diagnostic_cc_tests ()
|
||||
c_diagnostic_cc_tests ()
|
||||
{
|
||||
test_print_escaped_string ();
|
||||
test_print_parseable_fixits_none ();
|
||||
|
@ -324,7 +324,7 @@ test_opt_result_failure_at (const line_table_case &case_)
|
||||
/* Run all of the selftests within this file. */
|
||||
|
||||
void
|
||||
opt_problem_cc_tests ()
|
||||
c_opt_problem_cc_tests ()
|
||||
{
|
||||
test_opt_result_success ();
|
||||
for_each_line_table_case (test_opt_result_failure_at);
|
||||
|
@ -76,7 +76,6 @@ selftest::run_tests ()
|
||||
json_cc_tests ();
|
||||
cgraph_cc_tests ();
|
||||
optinfo_emit_json_cc_tests ();
|
||||
opt_problem_cc_tests ();
|
||||
ordered_hash_map_tests_cc_tests ();
|
||||
splay_tree_cc_tests ();
|
||||
|
||||
@ -95,7 +94,6 @@ selftest::run_tests ()
|
||||
/* Higher-level tests, or for components that other selftests don't
|
||||
rely on. */
|
||||
diagnostic_show_locus_cc_tests ();
|
||||
diagnostic_cc_tests ();
|
||||
diagnostic_format_json_cc_tests ();
|
||||
edit_context_cc_tests ();
|
||||
fold_const_cc_tests ();
|
||||
|
@ -222,7 +222,6 @@ extern void attribs_cc_tests ();
|
||||
extern void bitmap_cc_tests ();
|
||||
extern void cgraph_cc_tests ();
|
||||
extern void convert_cc_tests ();
|
||||
extern void diagnostic_cc_tests ();
|
||||
extern void diagnostic_format_json_cc_tests ();
|
||||
extern void diagnostic_show_locus_cc_tests ();
|
||||
extern void digraph_cc_tests ();
|
||||
@ -238,7 +237,6 @@ extern void hash_map_tests_cc_tests ();
|
||||
extern void hash_set_tests_cc_tests ();
|
||||
extern void input_cc_tests ();
|
||||
extern void json_cc_tests ();
|
||||
extern void opt_problem_cc_tests ();
|
||||
extern void optinfo_emit_json_cc_tests ();
|
||||
extern void opts_cc_tests ();
|
||||
extern void ordered_hash_map_tests_cc_tests ();
|
||||
|
Loading…
Reference in New Issue
Block a user