public inbox for gdb-testers@sourceware.org
help / color / mirror / Atom feed
* [binutils-gdb] Code cleanup: compile: func_addr -> func_sym
@ 2015-05-16 16:31 sergiodj+buildbot
  2015-05-16 16:31 ` Failures on Debian-i686, branch master sergiodj+buildbot
                   ` (9 more replies)
  0 siblings, 10 replies; 11+ messages in thread
From: sergiodj+buildbot @ 2015-05-16 16:31 UTC (permalink / raw)
  To: gdb-testers

*** TEST RESULTS FOR COMMIT 83d3415ef530c41af7e1ae98a7add97adb0cf5e0 ***

Author: Jan Kratochvil <jan.kratochvil@redhat.com>
Branch: master
Commit: 83d3415ef530c41af7e1ae98a7add97adb0cf5e0

Code cleanup: compile: func_addr -> func_sym
Currently the code fetches _gdb_expr address/types at multiple places, guessing
its parameters at multiple places etc.

Fetch it once, verify it has expected type and then rely on it.

While the patch tries to clean up the code it is still horrible due to the
missing C++ sub-classing.


gdb/ChangeLog
2015-05-16  Jan Kratochvil  <jan.kratochvil@redhat.com>

	* compile/compile-object-load.c (get_regs_type): Add parameter func_sym.
	Rely on its parameter count.
	(compile_object_load): Replace lookup_minimal_symbol_text by
	lookup_global_symbol_from_objfile.  Verify FUNC_SYM.  Set it in the
	return value.
	* compile/compile-object-load.h (struct compile_module): Replace
	func_addr by func_sym.
	* compile/compile-object-run.c: Include block.h.
	(compile_object_run): Reset module variable after it is freed.  Use
	FUNC_SYM instead of FUNC_ADDR.  Rely on it.


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

end of thread, other threads:[~2015-05-17  6:32 UTC | newest]

Thread overview: 11+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2015-05-16 16:31 [binutils-gdb] Code cleanup: compile: func_addr -> func_sym sergiodj+buildbot
2015-05-16 16:31 ` Failures on Debian-i686, branch master sergiodj+buildbot
2015-05-16 17:11 ` Failures on Debian-i686-native-extended-gdbserver, " sergiodj+buildbot
2015-05-16 20:02 ` Failures on Debian-x86_64-native-gdbserver-m64, " sergiodj+buildbot
2015-05-16 20:35 ` Failures on Fedora-i686, " sergiodj+buildbot
2015-05-16 21:06 ` Failures on Fedora-x86_64-native-extended-gdbserver-m32, " sergiodj+buildbot
2015-05-16 21:42 ` Failures on Fedora-x86_64-native-extended-gdbserver-m64, " sergiodj+buildbot
2015-05-16 22:11 ` Failures on Fedora-x86_64-cxx-build-m64, " sergiodj+buildbot
2015-05-16 22:14 ` Failures on Fedora-x86_64-m64, " sergiodj+buildbot
2015-05-17  5:29 ` Failures on Fedora-ppc64le-native-extended-gdbserver-m64, " sergiodj+buildbot
2015-05-17  6:32 ` Failures on Fedora-ppc64le-native-gdbserver-m64, " sergiodj+buildbot

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).