Mirror of the gdb-patches mailing list
 help / color / mirror / Atom feed
* [PATCH] [gdb/testsuite] Convert gdb.dwarf2/dw2-strp.exp to dwarf assembly
@ 2026-06-08 18:20 Tom de Vries
  2026-06-11 19:07 ` Guinevere Larsen
  0 siblings, 1 reply; 3+ messages in thread
From: Tom de Vries @ 2026-06-08 18:20 UTC (permalink / raw)
  To: gdb-patches

Convert gdb.dwarf2/dw2-strp.exp to dwarf assembly using
gdb/contrib/dwarf-to-dwarf-assembler.py.

Tested on x86_64-linux.
---
 gdb/testsuite/gdb.dwarf2/dw2-strp.S   | 187 --------------------------
 gdb/testsuite/gdb.dwarf2/dw2-strp.exp |  65 ++++++++-
 2 files changed, 59 insertions(+), 193 deletions(-)
 delete mode 100644 gdb/testsuite/gdb.dwarf2/dw2-strp.S

diff --git a/gdb/testsuite/gdb.dwarf2/dw2-strp.S b/gdb/testsuite/gdb.dwarf2/dw2-strp.S
deleted file mode 100644
index 69a790f036b..00000000000
--- a/gdb/testsuite/gdb.dwarf2/dw2-strp.S
+++ /dev/null
@@ -1,187 +0,0 @@
-/* This testcase is part of GDB, the GNU debugger.
-
-   Copyright 2008-2026 Free Software Foundation, Inc.
-
-   This program is free software; you can redistribute it and/or modify
-   it under the terms of the GNU General Public License as published by
-   the Free Software Foundation; either version 3 of the License, or
-   (at your option) any later version.
-
-   This program is distributed in the hope that it will be useful,
-   but WITHOUT ANY WARRANTY; without even the implied warranty of
-   MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
-   GNU General Public License for more details.
-
-   You should have received a copy of the GNU General Public License
-   along with this program.  If not, see <http://www.gnu.org/licenses/>.  */
-
-/* Debug information */
-
-	.section	.debug_info
-.Lcu1_begin:
-	.4byte		.Lcu1_end - .Lcu1_start	/* Length of Compilation Unit */
-.Lcu1_start:
-	.2byte		2			/* DWARF version number */
-	.4byte		.Ldebug_abbrev0		/* Offset Into Abbrev. Section */
-	.byte		4			/* Pointer Size (in bytes) */
-
-	/* CU die */
-	.uleb128	1			/* Abbrev: DW_TAG_compile_unit */
-	.4byte		.Lproducer		/* DW_AT_producer */
-	/* Use C++ to exploit a bug in parsing DW_AT_name "".  */
-	.byte		4			/* DW_AT_language (C++) -  */
-
-.Larray_type:
-	.uleb128	2			/* Abbrev: DW_TAG_array_type */
-	.4byte		.Lchar_type-.Lcu1_begin	/* DW_AT_type */
-
-	.uleb128	3			/* Abbrev: DW_TAG_subrange_type */
-	.4byte		.Luint_type-.Lcu1_begin	/* DW_AT_type */
-	.byte		0xd			/* DW_AT_upper_bound */
-	.byte		0			/* End of children of die */
-
-.Luint_type:
-	.uleb128	4			/* Abbrev: DW_TAG_base_type */
-	.4byte		.Luint_str		/* DW_AT_name */
-	.byte		4			/* DW_AT_byte_size */
-	.byte		7			/* DW_AT_encoding */
-
-.Lchar_type:
-	.uleb128	4			/* Abbrev: DW_TAG_base_type */
-	.4byte		.Lchar_str		/* DW_AT_name */
-	.byte		1			/* DW_AT_byte_size */
-	.byte		6			/* DW_AT_encoding */
-
-	.uleb128	5			/* Abbrev: DW_TAG_variable DW_FORM_strp */
-	.4byte		.Lvarname		/* DW_AT_name */
-	.4byte		.Lconst_type-.Lcu1_begin/* DW_AT_type */
-	.4byte		.Lvarcontents		/* DW_AT_const_value */
-	.byte		1			/* DW_AT_external */
-
-	.uleb128	6			/* Abbrev: DW_TAG_variable DW_FORM_string */
-	.string		"a_string2"		/* DW_AT_name */
-	.4byte		.Lconst_type-.Lcu1_begin/* DW_AT_type */
-	.string		"hello world2\n"	/* DW_AT_const_value */
-	.byte		1			/* DW_AT_external */
-
-.Lconst_type:
-	.uleb128	7			/* Abbrev: DW_TAG_const_type */
-	.4byte		.Larray_type-.Lcu1_begin/* DW_AT_type */
-
-	.uleb128	8			/* Abbrev: DW_TAG_variable (name "") */
-	.4byte		.Lemptyname		/* DW_AT_name */
-
-	.byte		0			/* End of children of CU */
-.Lcu1_end:
-
-	.section .debug_abbrev
-.Ldebug_abbrev0:
-	.uleb128	1			/* Abbrev code */
-	.uleb128	0x11			/* DW_TAG_compile_unit */
-	.byte		0x1			/* has_children */
-	.uleb128	0x25			/* DW_AT_producer */
-	.uleb128	0xe			/* DW_FORM_strp */
-	.uleb128	0x13			/* DW_AT_language */
-	.uleb128	0xb			/* DW_FORM_data1 */
-	.byte		0x0			/* Terminator */
-	.byte		0x0			/* Terminator */
-
-	.uleb128	2			/* Abbrev code */
-	.uleb128	0x1			/* TAG: DW_TAG_array_type */
-	.byte		0x1			/* DW_children_yes */
-	.uleb128	0x49			/* DW_AT_type */
-	.uleb128	0x13			/* DW_FORM_ref4 */
-	.byte		0x0			/* Terminator */
-	.byte		0x0			/* Terminator */
-
-	.uleb128	3			/* Abbrev code */
-	.uleb128	0x21			/* DW_TAG_subrange_type */
-	.byte		0x0			/* no children */
-	.uleb128	0x49			/* DW_AT_type */
-	.uleb128	0x13			/* DW_FORM_ref4 */
-	.uleb128	0x2f			/* DW_AT_upper_bound */
-	.uleb128	0xb			/* DW_FORM_data1 */
-	.byte		0x0			/* Terminator */
-	.byte		0x0			/* Terminator */
-
-	.uleb128	4			/* Abbrev code */
-	.uleb128	0x24			/* DW_TAG_base_type */
-	.byte		0x0			/* no_children */
-	.uleb128	0x3			/* DW_AT_name */
-	.uleb128	0xe			/* DW_FORM_strp */
-	.uleb128	0xb			/* DW_AT_byte_size */
-	.uleb128	0xb			/* DW_FORM_data1 */
-	.uleb128	0x3e			/* DW_AT_encoding */
-	.uleb128	0xb			/* DW_FORM_data1 */
-	.byte		0x0			/* Terminator */
-	.byte		0x0			/* Terminator */
-
-	.uleb128	5			/* Abbrev code */
-	.uleb128	0x34			/* DW_TAG_variable */
-	.byte		0x0			/* no_children */
-	.uleb128	0x3			/* DW_AT_name */
-	.uleb128	0xe			/* DW_FORM_strp */
-	.uleb128	0x49			/* DW_AT_type */
-	.uleb128	0x13			/* DW_FORM_ref4 */
-	.uleb128	0x1c			/* DW_AT_const_value */
-	.uleb128	0xe			/* DW_FORM_strp */
-	.uleb128	0x3f			/* DW_AT_external */
-	.uleb128	0xc			/* DW_FORM_flag */
-	.byte		0x0			/* Terminator */
-	.byte		0x0			/* Terminator */
-
-	.uleb128	6			/* Abbrev code */
-	.uleb128	0x34			/* DW_TAG_variable */
-	.byte		0x0			/* no_children */
-	.uleb128	0x3			/* DW_AT_name */
-	.uleb128	0x8			/* DW_FORM_string */
-	.uleb128	0x49			/* DW_AT_type */
-	.uleb128	0x13			/* DW_FORM_ref4 */
-	.uleb128	0x1c			/* DW_AT_const_value */
-	.uleb128	0x8			/* DW_FORM_string */
-	.uleb128	0x3f			/* DW_AT_external */
-	.uleb128	0xc			/* DW_FORM_flag */
-	.byte		0x0			/* Terminator */
-	.byte		0x0			/* Terminator */
-
-	.uleb128	7			/* Abbrev code */
-	.uleb128	0x26			/* DW_TAG_const_type */
-	.byte		0x0			/* DW_children_no */
-	.uleb128	0x49			/* DW_AT_type */
-	.uleb128	0x13			/* DW_FORM_ref4 */
-	.byte		0x0			/* Terminator */
-	.byte		0x0			/* Terminator */
-
-	.uleb128	8			/* Abbrev code */
-	.uleb128	0x34			/* DW_TAG_variable */
-	.byte		0x0			/* DW_children_no */
-	.uleb128	0x3			/* DW_AT_name */
-	.uleb128	0xe			/* DW_FORM_strp */
-	.byte		0x0			/* Terminator */
-	.byte		0x0			/* Terminator */
-
-	.byte		0x0			/* Terminator */
-
-/* String table */
-#ifdef __arm__
-	.section .debug_str,"MS",%progbits,1
-#else
-	.section .debug_str,"MS",@progbits,1
-#endif
-.Lproducer:
-	.string		"GNU C 3.3.3"
-.Lchar_str:
-	.string		"char"
-.Luint_str:
-	.string		"unsigned int"
-.Lvarname:
-	.string		"a_string"
-.Lvarcontents:
-	.string		"hello world!\n"
-.Lemptyname:
-	.string		""
-#ifdef __arm__
-	.section	.note.GNU-stack,"",%progbits
-#else
-	.section	.note.GNU-stack,"",@progbits
-#endif
diff --git a/gdb/testsuite/gdb.dwarf2/dw2-strp.exp b/gdb/testsuite/gdb.dwarf2/dw2-strp.exp
index 7dbc94f57ca..991c6323197 100644
--- a/gdb/testsuite/gdb.dwarf2/dw2-strp.exp
+++ b/gdb/testsuite/gdb.dwarf2/dw2-strp.exp
@@ -20,15 +20,68 @@ load_lib dwarf.exp
 # This test can only be run on targets which support DWARF-2 and use gas.
 require dwarf2_support
 
-standard_testfile .S
+standard_testfile main.c .S
+
+set asm_file [standard_output_file $srcfile2]
+Dwarf::assemble $asm_file {
+    declare_labels char const_type array_type unsigned_int
+
+    cu {version 2} {
+	DW_TAG_compile_unit {
+	    DW_AT_producer {GNU C 3.3.3} DW_FORM_strp
+	    DW_AT_language @DW_LANG_C_plus_plus
+	} {
+	    array_type: DW_TAG_array_type {
+		DW_AT_type :$char
+	    } {
+		DW_TAG_subrange_type {
+		    DW_AT_type :$unsigned_int
+		    DW_AT_upper_bound 13 DW_FORM_data1
+		}
+	    }
+	    unsigned_int: DW_TAG_base_type {
+		DW_AT_name {unsigned int} DW_FORM_strp
+		DW_AT_byte_size 4 DW_FORM_data1
+		DW_AT_encoding @DW_ATE_unsigned
+	    }
+	    char: DW_TAG_base_type {
+		DW_AT_name char DW_FORM_strp
+		DW_AT_byte_size 1 DW_FORM_data1
+		DW_AT_encoding @DW_ATE_signed_char
+	    }
+	    DW_TAG_variable {
+		DW_AT_name a_string DW_FORM_strp
+		DW_AT_type :$const_type
+		DW_AT_const_value {hello world!\n} DW_FORM_strp
+		DW_AT_external 1 DW_FORM_flag
+	    }
+	    DW_TAG_variable {
+		DW_AT_name a_string2 DW_FORM_string
+		DW_AT_type :$const_type
+		DW_AT_const_value {hello world2\n} DW_FORM_string
+		DW_AT_external 1 DW_FORM_flag
+	    }
+	    const_type: DW_TAG_const_type {
+		DW_AT_type :$array_type
+	    }
+	    DW_TAG_variable {
+		DW_AT_name  DW_FORM_strp
+	    }
+	}
+    }
+}
 
 if {[prepare_for_testing_full "failed to prepare" \
-	 [list $testfile debug main.c debug $srcfile nodebug]]} {
+	 [list $testfile debug $srcfile nodebug $asm_file nodebug]]} {
     return
 }
 
-gdb_test "p a_string" " = \"hello world!\\\\n\""
-gdb_test "ptype a_string" "type = const char \\\[14\\\]"
+gdb_test "p a_string" \
+    [string_to_regexp { = "hello world!\n"}]
+gdb_test "ptype a_string" \
+    [string_to_regexp {type = const char [14]}]
 
-gdb_test "p a_string2" " = \"hello world2\\\\n\""
-gdb_test "ptype a_string2" "type = const char \\\[14\\\]"
+gdb_test "p a_string2" \
+    [string_to_regexp { = "hello world2\n"}]
+gdb_test "ptype a_string2" \
+    [string_to_regexp {type = const char [14]}]

base-commit: 748350ae9e20d4698527c57bb3d3d76a33242cab
-- 
2.51.0


^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: [PATCH] [gdb/testsuite] Convert gdb.dwarf2/dw2-strp.exp to dwarf assembly
  2026-06-08 18:20 [PATCH] [gdb/testsuite] Convert gdb.dwarf2/dw2-strp.exp to dwarf assembly Tom de Vries
@ 2026-06-11 19:07 ` Guinevere Larsen
  2026-06-14 17:12   ` Tom de Vries
  0 siblings, 1 reply; 3+ messages in thread
From: Guinevere Larsen @ 2026-06-11 19:07 UTC (permalink / raw)
  To: Tom de Vries, gdb-patches

On 6/8/26 3:20 PM, Tom de Vries wrote:
> Convert gdb.dwarf2/dw2-strp.exp to dwarf assembly using
> gdb/contrib/dwarf-to-dwarf-assembler.py.
>
> Tested on x86_64-linux.

Hi! Thanks for doing this!

I looked over the generated dwarf output using readelf and it seems that 
everything that is tested by the testcase is still exactly the same so 
this I think this is just an upgrade!

Reviewed-By: Guinevere Larsen <guinevere@redhat.com>

-- 
Cheers,
Guinevere Larsen
it/its
she/her (deprecated)

> ---
>   gdb/testsuite/gdb.dwarf2/dw2-strp.S   | 187 --------------------------
>   gdb/testsuite/gdb.dwarf2/dw2-strp.exp |  65 ++++++++-
>   2 files changed, 59 insertions(+), 193 deletions(-)
>   delete mode 100644 gdb/testsuite/gdb.dwarf2/dw2-strp.S
>
> diff --git a/gdb/testsuite/gdb.dwarf2/dw2-strp.S b/gdb/testsuite/gdb.dwarf2/dw2-strp.S
> deleted file mode 100644
> index 69a790f036b..00000000000
> --- a/gdb/testsuite/gdb.dwarf2/dw2-strp.S
> +++ /dev/null
> @@ -1,187 +0,0 @@
> -/* This testcase is part of GDB, the GNU debugger.
> -
> -   Copyright 2008-2026 Free Software Foundation, Inc.
> -
> -   This program is free software; you can redistribute it and/or modify
> -   it under the terms of the GNU General Public License as published by
> -   the Free Software Foundation; either version 3 of the License, or
> -   (at your option) any later version.
> -
> -   This program is distributed in the hope that it will be useful,
> -   but WITHOUT ANY WARRANTY; without even the implied warranty of
> -   MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
> -   GNU General Public License for more details.
> -
> -   You should have received a copy of the GNU General Public License
> -   along with this program.  If not, see <http://www.gnu.org/licenses/>.  */
> -
> -/* Debug information */
> -
> -	.section	.debug_info
> -.Lcu1_begin:
> -	.4byte		.Lcu1_end - .Lcu1_start	/* Length of Compilation Unit */
> -.Lcu1_start:
> -	.2byte		2			/* DWARF version number */
> -	.4byte		.Ldebug_abbrev0		/* Offset Into Abbrev. Section */
> -	.byte		4			/* Pointer Size (in bytes) */
> -
> -	/* CU die */
> -	.uleb128	1			/* Abbrev: DW_TAG_compile_unit */
> -	.4byte		.Lproducer		/* DW_AT_producer */
> -	/* Use C++ to exploit a bug in parsing DW_AT_name "".  */
> -	.byte		4			/* DW_AT_language (C++) -  */
> -
> -.Larray_type:
> -	.uleb128	2			/* Abbrev: DW_TAG_array_type */
> -	.4byte		.Lchar_type-.Lcu1_begin	/* DW_AT_type */
> -
> -	.uleb128	3			/* Abbrev: DW_TAG_subrange_type */
> -	.4byte		.Luint_type-.Lcu1_begin	/* DW_AT_type */
> -	.byte		0xd			/* DW_AT_upper_bound */
> -	.byte		0			/* End of children of die */
> -
> -.Luint_type:
> -	.uleb128	4			/* Abbrev: DW_TAG_base_type */
> -	.4byte		.Luint_str		/* DW_AT_name */
> -	.byte		4			/* DW_AT_byte_size */
> -	.byte		7			/* DW_AT_encoding */
> -
> -.Lchar_type:
> -	.uleb128	4			/* Abbrev: DW_TAG_base_type */
> -	.4byte		.Lchar_str		/* DW_AT_name */
> -	.byte		1			/* DW_AT_byte_size */
> -	.byte		6			/* DW_AT_encoding */
> -
> -	.uleb128	5			/* Abbrev: DW_TAG_variable DW_FORM_strp */
> -	.4byte		.Lvarname		/* DW_AT_name */
> -	.4byte		.Lconst_type-.Lcu1_begin/* DW_AT_type */
> -	.4byte		.Lvarcontents		/* DW_AT_const_value */
> -	.byte		1			/* DW_AT_external */
> -
> -	.uleb128	6			/* Abbrev: DW_TAG_variable DW_FORM_string */
> -	.string		"a_string2"		/* DW_AT_name */
> -	.4byte		.Lconst_type-.Lcu1_begin/* DW_AT_type */
> -	.string		"hello world2\n"	/* DW_AT_const_value */
> -	.byte		1			/* DW_AT_external */
> -
> -.Lconst_type:
> -	.uleb128	7			/* Abbrev: DW_TAG_const_type */
> -	.4byte		.Larray_type-.Lcu1_begin/* DW_AT_type */
> -
> -	.uleb128	8			/* Abbrev: DW_TAG_variable (name "") */
> -	.4byte		.Lemptyname		/* DW_AT_name */
> -
> -	.byte		0			/* End of children of CU */
> -.Lcu1_end:
> -
> -	.section .debug_abbrev
> -.Ldebug_abbrev0:
> -	.uleb128	1			/* Abbrev code */
> -	.uleb128	0x11			/* DW_TAG_compile_unit */
> -	.byte		0x1			/* has_children */
> -	.uleb128	0x25			/* DW_AT_producer */
> -	.uleb128	0xe			/* DW_FORM_strp */
> -	.uleb128	0x13			/* DW_AT_language */
> -	.uleb128	0xb			/* DW_FORM_data1 */
> -	.byte		0x0			/* Terminator */
> -	.byte		0x0			/* Terminator */
> -
> -	.uleb128	2			/* Abbrev code */
> -	.uleb128	0x1			/* TAG: DW_TAG_array_type */
> -	.byte		0x1			/* DW_children_yes */
> -	.uleb128	0x49			/* DW_AT_type */
> -	.uleb128	0x13			/* DW_FORM_ref4 */
> -	.byte		0x0			/* Terminator */
> -	.byte		0x0			/* Terminator */
> -
> -	.uleb128	3			/* Abbrev code */
> -	.uleb128	0x21			/* DW_TAG_subrange_type */
> -	.byte		0x0			/* no children */
> -	.uleb128	0x49			/* DW_AT_type */
> -	.uleb128	0x13			/* DW_FORM_ref4 */
> -	.uleb128	0x2f			/* DW_AT_upper_bound */
> -	.uleb128	0xb			/* DW_FORM_data1 */
> -	.byte		0x0			/* Terminator */
> -	.byte		0x0			/* Terminator */
> -
> -	.uleb128	4			/* Abbrev code */
> -	.uleb128	0x24			/* DW_TAG_base_type */
> -	.byte		0x0			/* no_children */
> -	.uleb128	0x3			/* DW_AT_name */
> -	.uleb128	0xe			/* DW_FORM_strp */
> -	.uleb128	0xb			/* DW_AT_byte_size */
> -	.uleb128	0xb			/* DW_FORM_data1 */
> -	.uleb128	0x3e			/* DW_AT_encoding */
> -	.uleb128	0xb			/* DW_FORM_data1 */
> -	.byte		0x0			/* Terminator */
> -	.byte		0x0			/* Terminator */
> -
> -	.uleb128	5			/* Abbrev code */
> -	.uleb128	0x34			/* DW_TAG_variable */
> -	.byte		0x0			/* no_children */
> -	.uleb128	0x3			/* DW_AT_name */
> -	.uleb128	0xe			/* DW_FORM_strp */
> -	.uleb128	0x49			/* DW_AT_type */
> -	.uleb128	0x13			/* DW_FORM_ref4 */
> -	.uleb128	0x1c			/* DW_AT_const_value */
> -	.uleb128	0xe			/* DW_FORM_strp */
> -	.uleb128	0x3f			/* DW_AT_external */
> -	.uleb128	0xc			/* DW_FORM_flag */
> -	.byte		0x0			/* Terminator */
> -	.byte		0x0			/* Terminator */
> -
> -	.uleb128	6			/* Abbrev code */
> -	.uleb128	0x34			/* DW_TAG_variable */
> -	.byte		0x0			/* no_children */
> -	.uleb128	0x3			/* DW_AT_name */
> -	.uleb128	0x8			/* DW_FORM_string */
> -	.uleb128	0x49			/* DW_AT_type */
> -	.uleb128	0x13			/* DW_FORM_ref4 */
> -	.uleb128	0x1c			/* DW_AT_const_value */
> -	.uleb128	0x8			/* DW_FORM_string */
> -	.uleb128	0x3f			/* DW_AT_external */
> -	.uleb128	0xc			/* DW_FORM_flag */
> -	.byte		0x0			/* Terminator */
> -	.byte		0x0			/* Terminator */
> -
> -	.uleb128	7			/* Abbrev code */
> -	.uleb128	0x26			/* DW_TAG_const_type */
> -	.byte		0x0			/* DW_children_no */
> -	.uleb128	0x49			/* DW_AT_type */
> -	.uleb128	0x13			/* DW_FORM_ref4 */
> -	.byte		0x0			/* Terminator */
> -	.byte		0x0			/* Terminator */
> -
> -	.uleb128	8			/* Abbrev code */
> -	.uleb128	0x34			/* DW_TAG_variable */
> -	.byte		0x0			/* DW_children_no */
> -	.uleb128	0x3			/* DW_AT_name */
> -	.uleb128	0xe			/* DW_FORM_strp */
> -	.byte		0x0			/* Terminator */
> -	.byte		0x0			/* Terminator */
> -
> -	.byte		0x0			/* Terminator */
> -
> -/* String table */
> -#ifdef __arm__
> -	.section .debug_str,"MS",%progbits,1
> -#else
> -	.section .debug_str,"MS",@progbits,1
> -#endif
> -.Lproducer:
> -	.string		"GNU C 3.3.3"
> -.Lchar_str:
> -	.string		"char"
> -.Luint_str:
> -	.string		"unsigned int"
> -.Lvarname:
> -	.string		"a_string"
> -.Lvarcontents:
> -	.string		"hello world!\n"
> -.Lemptyname:
> -	.string		""
> -#ifdef __arm__
> -	.section	.note.GNU-stack,"",%progbits
> -#else
> -	.section	.note.GNU-stack,"",@progbits
> -#endif
> diff --git a/gdb/testsuite/gdb.dwarf2/dw2-strp.exp b/gdb/testsuite/gdb.dwarf2/dw2-strp.exp
> index 7dbc94f57ca..991c6323197 100644
> --- a/gdb/testsuite/gdb.dwarf2/dw2-strp.exp
> +++ b/gdb/testsuite/gdb.dwarf2/dw2-strp.exp
> @@ -20,15 +20,68 @@ load_lib dwarf.exp
>   # This test can only be run on targets which support DWARF-2 and use gas.
>   require dwarf2_support
>   
> -standard_testfile .S
> +standard_testfile main.c .S
> +
> +set asm_file [standard_output_file $srcfile2]
> +Dwarf::assemble $asm_file {
> +    declare_labels char const_type array_type unsigned_int
> +
> +    cu {version 2} {
> +	DW_TAG_compile_unit {
> +	    DW_AT_producer {GNU C 3.3.3} DW_FORM_strp
> +	    DW_AT_language @DW_LANG_C_plus_plus
> +	} {
> +	    array_type: DW_TAG_array_type {
> +		DW_AT_type :$char
> +	    } {
> +		DW_TAG_subrange_type {
> +		    DW_AT_type :$unsigned_int
> +		    DW_AT_upper_bound 13 DW_FORM_data1
> +		}
> +	    }
> +	    unsigned_int: DW_TAG_base_type {
> +		DW_AT_name {unsigned int} DW_FORM_strp
> +		DW_AT_byte_size 4 DW_FORM_data1
> +		DW_AT_encoding @DW_ATE_unsigned
> +	    }
> +	    char: DW_TAG_base_type {
> +		DW_AT_name char DW_FORM_strp
> +		DW_AT_byte_size 1 DW_FORM_data1
> +		DW_AT_encoding @DW_ATE_signed_char
> +	    }
> +	    DW_TAG_variable {
> +		DW_AT_name a_string DW_FORM_strp
> +		DW_AT_type :$const_type
> +		DW_AT_const_value {hello world!\n} DW_FORM_strp
> +		DW_AT_external 1 DW_FORM_flag
> +	    }
> +	    DW_TAG_variable {
> +		DW_AT_name a_string2 DW_FORM_string
> +		DW_AT_type :$const_type
> +		DW_AT_const_value {hello world2\n} DW_FORM_string
> +		DW_AT_external 1 DW_FORM_flag
> +	    }
> +	    const_type: DW_TAG_const_type {
> +		DW_AT_type :$array_type
> +	    }
> +	    DW_TAG_variable {
> +		DW_AT_name  DW_FORM_strp
> +	    }
> +	}
> +    }
> +}
>   
>   if {[prepare_for_testing_full "failed to prepare" \
> -	 [list $testfile debug main.c debug $srcfile nodebug]]} {
> +	 [list $testfile debug $srcfile nodebug $asm_file nodebug]]} {
>       return
>   }
>   
> -gdb_test "p a_string" " = \"hello world!\\\\n\""
> -gdb_test "ptype a_string" "type = const char \\\[14\\\]"
> +gdb_test "p a_string" \
> +    [string_to_regexp { = "hello world!\n"}]
> +gdb_test "ptype a_string" \
> +    [string_to_regexp {type = const char [14]}]
>   
> -gdb_test "p a_string2" " = \"hello world2\\\\n\""
> -gdb_test "ptype a_string2" "type = const char \\\[14\\\]"
> +gdb_test "p a_string2" \
> +    [string_to_regexp { = "hello world2\n"}]
> +gdb_test "ptype a_string2" \
> +    [string_to_regexp {type = const char [14]}]
>
> base-commit: 748350ae9e20d4698527c57bb3d3d76a33242cab


^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: [PATCH] [gdb/testsuite] Convert gdb.dwarf2/dw2-strp.exp to dwarf assembly
  2026-06-11 19:07 ` Guinevere Larsen
@ 2026-06-14 17:12   ` Tom de Vries
  0 siblings, 0 replies; 3+ messages in thread
From: Tom de Vries @ 2026-06-14 17:12 UTC (permalink / raw)
  To: Guinevere Larsen, gdb-patches

On 6/11/26 9:07 PM, Guinevere Larsen wrote:
> On 6/8/26 3:20 PM, Tom de Vries wrote:
>> Convert gdb.dwarf2/dw2-strp.exp to dwarf assembly using
>> gdb/contrib/dwarf-to-dwarf-assembler.py.
>>
>> Tested on x86_64-linux.
> 
> Hi! Thanks for doing this!
> 
> I looked over the generated dwarf output using readelf and it seems that 
> everything that is tested by the testcase is still exactly the same so 
> this I think this is just an upgrade!
> 

Hi Gwen,

thanks for the review, I've pushed this.

- Tom

> Reviewed-By: Guinevere Larsen <guinevere@redhat.com>
> 


^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2026-06-14 17:12 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2026-06-08 18:20 [PATCH] [gdb/testsuite] Convert gdb.dwarf2/dw2-strp.exp to dwarf assembly Tom de Vries
2026-06-11 19:07 ` Guinevere Larsen
2026-06-14 17:12   ` Tom de Vries

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox