This is the mail archive of the mailing list for the GDB project.

Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

Re: [patch][python] 2/3 Python representation of GDB line tables (Testsuite)

On 23/10/13 21:53, Tom Tromey wrote:
>>>>>> "Phil" == Phil Muldoon <> writes:
> Phil> This patch in the series contains the testsuite code for Python
> Phil> linetables.
> Phil> diff --git a/gdb/testsuite/gdb.python/py-linetable.S
> Phil> b/gdb/testsuite/gdb.python/py-linetable.S
> Phil> new file mode 100644
> Phil> index 0000000..5957222
> Phil> --- /dev/null
> Phil> +++ b/gdb/testsuite/gdb.python/py-linetable.S
> Phil> @@ -0,0 +1,345 @@
> Phil> +	.file	"py-linetable.c"
> Phil> +	.text
> Phil> +.Ltext0:
> Normally we hack in a comment header at the top.
> I wonder whether assembly is truly needed in this case.

It was included at Jan's suggestion.  To ensure compatibility between
compiler versions. However, the test is compiled unoptimized so I
suspect the potential of change in the linetable between versions is
slim.  I have no strong opinion.  What do you think?

> Phil> +#
> Phil> +# You should have received a copy of the GNU General Public License
> Phil> +# along with this program.  If not, see <>.
> Phil> +load_lib gdb-python.exp
> Phil> +set opts {}
> Phil> +standard_testfile .S
> Blank line between comments and code.
> Phil> +gdb_py_test_silent_cmd "python lt =
> Phil> gdb.selected_frame().find_sal().symtab.linetable()" \
> Your patch got wrapped.

No idea how this happened. It was produced by git format-patch, and
the file inserted into the email (IE, not copy and pasted).  So that
was weird.



Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]