This is the mail archive of the
gdb-patches@sourceware.org
mailing list for the GDB project.
Re: RFA: probable rs6000-aix-tdep.c bug found by clang
> >> * rs6000-aix-tdep.c (rs6000_aix_osabi_sniffer): Remove extraneous
> >> semicolon.
>
> I suspect this is the root cause of PR8966 (AIX 5.1 single_step configuration
> is broken). http://sourceware.org/bugzilla/show_bug.cgi?id=8966
>
> We're always returning the _UNKNOWN osabi, so this is unreachable:
On the contrary, I think we always returned the AIX osabi.
> > So, isn't rs6000_aix_osabi_sniffer going to be called if, and only
> > if, the bfd has a bfd_target_xcoff_flavour, thus making the check
> > superfluous?
>
> Sure looks like it.
OK - I will take care of removing the condition and simplifying
the function.
--
Joel