gdb/testsuite: make declare_labels use better default label names
authorSimon Marchi <simon.marchi@efficios.com>
Fri, 4 Dec 2020 20:08:54 +0000 (15:08 -0500)
committerSimon Marchi <simon.marchi@efficios.com>
Fri, 4 Dec 2020 20:08:54 +0000 (15:08 -0500)
commite633b1172a5cf5d099644474e03f9228dc870b25
tree4a43014e2e91b2b8f37c96f430976c1aaf838e85
parentab954e4a53cab8c33728e6708695abc1616c90b1
gdb/testsuite: make declare_labels use better default label names

When using the single-element form of argument to declare_labels, the
generated label (in the assembly file) is of the format ".LlabelN",
where N is a number.

I propose making it use the name of the label by default.  Calling:

    declare_labels foo

will generate the ".LfooN" in the assembly file (again, where N is a
number).  When debugging the output of the DWARF assembler, it makes it
easier to map labels to the source.  Also, when defining the same label
twice by mistake in the Tcl code (like I d id), it's easier to track the
error from the message to the root cause:

    -/home/smarchi/build/binutils-gdb/gdb/testsuite/outputs/gdb.dwarf2/implptrpiece/implptrpiece-dw.S:62: Error: symbol `.Llabel5' is already defined
    +/home/smarchi/build/binutils-gdb/gdb/testsuite/outputs/gdb.dwarf2/implptrpiece/implptrpiece-dw.S:62: Error: symbol `.Lvar_label5' is already defined

This doesn't change anything for the test cases, it just makes the
assembly output a bit nicer.

gdb/testsuite/ChangeLog:

* lib/dwarf.exp (declare_labels): Use name as text if text is
not provided.

Change-Id: I63856c1fa6390498fd5b9d66f471f817ff0a465c
gdb/testsuite/ChangeLog
gdb/testsuite/lib/dwarf.exp