This is the mail archive of the gdb-patches@sourceware.org mailing list for the GDB project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

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


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]