From beb4b03cbebf97e3f16836a770c8eced22ee6a1c Mon Sep 17 00:00:00 2001 From: Joel Brobecker Date: Fri, 19 Oct 2012 19:58:18 +0000 Subject: ppc-aix osabi sniffer: Turn test of bfd flavour into assertion Due to the way this function is registers, we know that given bfd's flavour should always be bfd_target_xcoff_flavour, thus making the former test always true, which means that this function should always return GDB_OSABI_AIX, and never return GDB_OSABI_UNKNOWN. This patch also fixes a typo detected by Tom Tromey that caused the test itself to be completely ineffective. gdb/ChangeLog (by Tom Tromey and Joel Brobecker): * rs6000-aix-tdep.c (rs6000_aix_osabi_sniffer): Replace inneffective if condition by gdb assertion. Add function description comment. --- gdb/rs6000-aix-tdep.c | 13 +++++++++---- 1 file changed, 9 insertions(+), 4 deletions(-) (limited to 'gdb/rs6000-aix-tdep.c') diff --git a/gdb/rs6000-aix-tdep.c b/gdb/rs6000-aix-tdep.c index 59cfa73..06b43de 100644 --- a/gdb/rs6000-aix-tdep.c +++ b/gdb/rs6000-aix-tdep.c @@ -719,14 +719,19 @@ rs6000_software_single_step (struct frame_info *frame) return 1; } +/* Implement an osabi sniffer for RS6000/AIX. + + This function assumes that ABFD's flavour is XCOFF. In other words, + it should be registered as a sniffer for bfd_target_xcoff_flavour + objfiles only. A failed assertion will be raised if this condition + is not met. */ + static enum gdb_osabi rs6000_aix_osabi_sniffer (bfd *abfd) { - - if (bfd_get_flavour (abfd) == bfd_target_xcoff_flavour); - return GDB_OSABI_AIX; + gdb_assert (bfd_get_flavour (abfd) == bfd_target_xcoff_flavour); - return GDB_OSABI_UNKNOWN; + return GDB_OSABI_AIX; } static void -- cgit v1.1