Re: [PATCH] dwarf: Add -gdwarf{32,64} options

2020-12-02 Thread Jakub Jelinek via Gcc-patches
On Tue, Dec 01, 2020 at 01:48:31PM +0100, Jakub Jelinek via Gcc-patches wrote: > I guess it depends on whether for 32-bit target and -gdwarf64, when > calling dw2_assemble_integer with non-CONST_INT argument we only > need positive values or might need negative ones too. > Because positive ones can

Re: [PATCH] dwarf: Add -gdwarf{32,64} options

2020-12-01 Thread Jakub Jelinek via Gcc-patches
On Tue, Dec 01, 2020 at 01:03:07PM +0100, Richard Biener wrote: > > So yes, we can either do a sorry, error, or could just avoid 64-bit > > relocations (depending on endianity instead of emitting > > .quad expression_that_needs_relocation > > emit > > .long expression_that_needs_relocation, 0 > > o

Re: [PATCH] dwarf: Add -gdwarf{32,64} options

2020-12-01 Thread Richard Biener
On Tue, 1 Dec 2020, Jakub Jelinek wrote: > On Tue, Dec 01, 2020 at 11:49:43AM +0100, Richard Biener wrote: > > On Tue, 1 Dec 2020, Jakub Jelinek wrote: > > > > > Hi! > > > > > > The following patch makes the choice between 32-bit and 64-bit DWARF > > > formats > > > selectable by command line s

Re: [PATCH] dwarf: Add -gdwarf{32,64} options

2020-12-01 Thread Jakub Jelinek via Gcc-patches
On Tue, Dec 01, 2020 at 11:49:43AM +0100, Richard Biener wrote: > On Tue, 1 Dec 2020, Jakub Jelinek wrote: > > > Hi! > > > > The following patch makes the choice between 32-bit and 64-bit DWARF formats > > selectable by command line switch, rather than being hardcoded through > > DWARF_OFFSET_SIZ

Re: [PATCH] dwarf: Add -gdwarf{32,64} options

2020-12-01 Thread Richard Biener
On Tue, 1 Dec 2020, Jakub Jelinek wrote: > Hi! > > The following patch makes the choice between 32-bit and 64-bit DWARF formats > selectable by command line switch, rather than being hardcoded through > DWARF_OFFSET_SIZE macro. > > The options themselves don't turn on debug info themselves, so o

[PATCH] dwarf: Add -gdwarf{32,64} options

2020-12-01 Thread Jakub Jelinek via Gcc-patches
Hi! The following patch makes the choice between 32-bit and 64-bit DWARF formats selectable by command line switch, rather than being hardcoded through DWARF_OFFSET_SIZE macro. The options themselves don't turn on debug info themselves, so one needs to use -g -gdwarf64 or similar. Ok for trunk i