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: Remove code handling old ARM aliases from GDB


On Thu, 5 May 2011, Hans-Peter Nilsson wrote:

> > -if { [istarget xscale*-*-*] } {
> > +if { [istarget arm*-*-*] } {

> How did you test these changes?

I considered them sufficiently obviously syntactically correct not to need 
testing.

> These changes broke testing for arm-elfo, on ang x86_64 host
> using current binutils CVS like so:

> FAIL: xscale wcmpgt.cgs (execution)
> FAIL: xscale wmac.cgs (execution)
> FAIL: xscale wsra.cgs (execution)

> WARNING: program timed out.
> FAIL: xscale blx.cgs (execution)

I don't call that breaking testing; I call that exposing bugs (whether in 
the simulator or testsuite) that were previously hidden by the use of an 
obsolete target triplet name.  Since the simulator features do not depend 
on the target triplet name, it seems clear that these tests should be run 
for all ARM targets - and so it seems better to expose the problems by 
using a current target name than to make the formally no-op change of 
using if { 0 } to replace an always-false conditional.

If the issue were actually missing assembler or simulator options a lot 
more tests in those directories would fail, so it seems much more likely 
that the relevant simulator functionality has become bitrotten over time.

-- 
Joseph S. Myers
joseph@codesourcery.com


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