Re: TPF: disable discriminators

2012-07-31 Thread Richard Henderson
On 2012-07-30 21:07, DJ Delorie wrote: + +/* GAS supports it, but the debuggers don't, so avoid it. */ +#define SUPPORTS_DISCRIMINATOR 0 Then you shouldn't be fiddling this, but rather dwarf_strict. See e.g. darwin and vxworks ports. r~

Re: TPF: disable discriminators

2012-07-31 Thread DJ Delorie
Ah, the original complaint was for a gcc branch which doesn't have your strict-dwarf/discriminator patch. How's this? Index: gcc/config/s390/s390.c === --- gcc/config/s390/s390.c (revision 190017) +++ gcc/config/s390/s390.c

Re: TPF: disable discriminators

2012-07-31 Thread DJ Delorie
TARGET_TPF is always defined. Just use a C if. Otherwise ok. Thanks, checked in as attached. What about older branches? 4.7 needs this patch, 4.6 needs my original patch. 2012-07-31 DJ Delorie d...@redhat.com * config/s390/s390.c (s390_option_override): Disable DWARF 3/4

Re: TPF: disable discriminators

2012-07-31 Thread Richard Henderson
On 2012-07-31 15:51, DJ Delorie wrote: I don't see that 4.6 requires a different patch. 4.6 is missing this: 2011-04-01 Richard Henderson r...@redhat.com PR 48400 * dwarf2out.c (dwarf2out_source_line): Disable discriminators in strict mode before dwarf4. Re-order

Re: TPF: disable discriminators

2012-07-31 Thread DJ Delorie
I don't see that 4.6 requires a different patch. 4.6 is missing this: 2011-04-01 Richard Henderson r...@redhat.com PR 48400 * dwarf2out.c (dwarf2out_source_line): Disable discriminators in strict mode before dwarf4. Re-order tests to early out before

TPF: disable discriminators

2012-07-30 Thread DJ Delorie
The TPF assembler supports dwarf4 discriminators, but the TPF debuggers do not. Ok to apply? * config/s390/tpf.h (SUPPORTS_DISCRIMINATOR): Define to 0 for TPF. Index: gcc/config/s390/tpf.h === --- gcc/config/s390/tpf.h