Home Original page

gas/Dwarf: improve debug info generation from .irp and alike blocks

Jan Beulich jbeulich@suse.com
Fri Mar 25 08:47:00 GMT 2022 
On 25.03.2022 07:42, Alan Modra via Binutils wrote:
> am33_2.0-linux is a mn10300 target.
> 
> 	* testsuite/gas/elf/dwarf-5-irp.d: xfail am33.

Thanks for noticing and fixing. I (mistakenly) had dropped this purposefully
from the list of xfails found in the pre-existing sibling test, for not
finding any respective source files in bfd/, and hence assuming this was a
previously dropped architecture. But clearly I should have looked at
config.bfd as well.

Iirc I did raise the question before: Could we have a file in the repo
listing targets that one should include when testing changes affecting most
(or all) targets? At present I'm trying to maintain an ad hoc list locally,
somewhat sub-categorized further by object formats.

As an aside - there looks to be further fallout from this change of mine,
partly already hinted at by post-commit-message comments in the submission
(but after some more thinking about it, the effects are worse than I first
thought when dealing with compiler output rather than real assembly source
files). For other than .macro, I'm intending to actually do what
approximately I did say there and insert a .linefile first thing in the
template subsequently used for expansion. For .macro I'm presently thinking
of some less pretty solution, but I'll have to experiment some more - the
main issue there is that I'd want to avoid breaking the PR gas/16908
workaround (but subsequently I may want to add some heuristics there, and I
have a yet to be submitted [after a full round of testing] fix to that
workaround as well).

Jan

> diff --git a/gas/testsuite/gas/elf/dwarf-5-irp.d b/gas/testsuite/gas/elf/dwarf-5-irp.d
> index e7bdda70e16..c305c8dc276 100644
> --- a/gas/testsuite/gas/elf/dwarf-5-irp.d
> +++ b/gas/testsuite/gas/elf/dwarf-5-irp.d
> @@ -1,11 +1,11 @@
>  #as: --gdwarf-5
>  #name: line number entries for section changes inside .irp
>  #readelf: -W -wlrR
> -# The cr16 crx ft32 mn10* msp430 nds32* and rl78 targets do not evaluate the subtraction of symbols at assembly time.
> +# The am33 cr16 crx ft32 mn10* msp430 nds32* and rl78 targets do not evaluate the subtraction of symbols at assembly time.
>  # The bfin target does not allow .subsection with an equated symbol as operand.
>  # The d30v target emits sufficiently different debug info, apparently also covering padding it inserts.
>  # The riscv targets do not support the subtraction of symbols.
> -#xfail: bfin-* cr16-* crx-* d30v-* ft32-* mn10*-* msp430-* nds32*-* riscv*-* rl78-*
> +#xfail: am33*-* bfin-* cr16-* crx-* d30v-* ft32-* mn10*-* msp430-* nds32*-* riscv*-* rl78-*
>  
>  Raw dump of debug contents .*
>  #...
>
More information about the Binutils mailing list