diff options
author | Alan Hayward <alan.hayward@arm.com> | 2019-07-04 11:48:16 +0100 |
---|---|---|
committer | Alan Hayward <alan.hayward@arm.com> | 2019-07-04 11:55:20 +0100 |
commit | 2b40fda74b2af11a914d50f4e54c1472e1bd81fb (patch) | |
tree | 279bb13c5cd574f712dd72f46a03bb0341059929 /gdb/amd64-linux-tdep.c | |
parent | d4693039f950eefb983a66e3270209b738e4ce8a (diff) | |
download | gdb-2b40fda74b2af11a914d50f4e54c1472e1bd81fb.zip gdb-2b40fda74b2af11a914d50f4e54c1472e1bd81fb.tar.gz gdb-2b40fda74b2af11a914d50f4e54c1472e1bd81fb.tar.bz2 |
i386/AArch64: Remove old xml tests
Both the i386, X86_64 and AArch64 builds of gdbserver include a bunch of legacy
xml files, dat files and auto generated C files, when building for unit test.
These tests exists back from when feature target descriptions were added to
prove that the new target descriptions were identical to the original
older versions. The old files are not used for anything other than these tests.
Now that this has been proven, we are not gaining anything by keeping the
original files and tests. Should new functionality be added, it would break
the tests, unless the functionality was backported to the xml. There is no
requirement that we must match the exact xml from N releases ago. It adds
obfuscation, where as the feature target descriptions were meant to simplify
the code.
In addition, there are a bunch of xml and dat files which are completely unused.
This patch removes the selftests and the target descriptions from gdbserver.
Update the unittest to allow 0 tests (note, this failed on other targets that
never had any tests).
gdb/ChangeLog:
* aarch64-tdep.c: Remove xml self tests.
* amd64-linux-tdep.c: Likewise.
* amd64-tdep.c: Likewise.
* i386-linux-tdep.c: Likewise.
* i386-tdep.c: Likewise.
gdb/gdbserver/ChangeLog:
* configure.srv: Remove legacy xml.
* linux-aarch64-low.c (initialize_low_arch): Remove
initialize_low_tdesc call.
* linux-aarch64-tdesc-selftest.c: Remove file.
* linux-aarch64-tdesc.h (initialize_low_tdesc): Remove.
* linux-x86-low.c (initialize_low_arch): Remove
initialize_low_tdesc call.
* linux-x86-tdesc-selftest.c: Remove file.
* linux-x86-tdesc.h (initialize_low_tdesc): Remove.
gdb/testsuite/ChangeLog:
* gdb.server/unittest.exp: Allow 0 unit tests to run.
Diffstat (limited to 'gdb/amd64-linux-tdep.c')
-rw-r--r-- | gdb/amd64-linux-tdep.c | 28 |
1 files changed, 0 insertions, 28 deletions
diff --git a/gdb/amd64-linux-tdep.c b/gdb/amd64-linux-tdep.c index 5475cf62..008817c 100644 --- a/gdb/amd64-linux-tdep.c +++ b/gdb/amd64-linux-tdep.c @@ -2279,32 +2279,4 @@ _initialize_amd64_linux_tdep (void) GDB_OSABI_LINUX, amd64_linux_init_abi); gdbarch_register_osabi (bfd_arch_i386, bfd_mach_x64_32, GDB_OSABI_LINUX, amd64_x32_linux_init_abi); - -#if GDB_SELF_TEST - struct - { - const char *xml; - uint64_t mask; - } xml_masks[] = { - { "i386/amd64-linux.xml", X86_XSTATE_SSE_MASK }, - { "i386/amd64-avx-linux.xml", X86_XSTATE_AVX_MASK }, - { "i386/amd64-mpx-linux.xml", X86_XSTATE_MPX_MASK }, - { "i386/amd64-avx-mpx-linux.xml", X86_XSTATE_AVX_MPX_MASK }, - { "i386/amd64-avx-avx512-linux.xml", X86_XSTATE_AVX_AVX512_MASK }, - { "i386/amd64-avx-mpx-avx512-pku-linux.xml", - X86_XSTATE_AVX_MPX_AVX512_PKU_MASK }, - { "i386/x32-linux.xml", X86_XSTATE_SSE_MASK }, - { "i386/x32-avx-linux.xml", X86_XSTATE_AVX_MASK }, - { "i386/x32-avx-avx512-linux.xml", X86_XSTATE_AVX_AVX512_MASK }, - }; - - for (auto &a : xml_masks) - { - auto tdesc = amd64_linux_read_description (a.mask, - startswith (a.xml, - "i386/x32")); - - selftests::record_xml_tdesc (a.xml, tdesc); - } -#endif /* GDB_SELF_TEST */ } |