New GDB 7.5.x release sometime next week?

New GDB 7.5.x release sometime next week?

[Joel Brobecker]

[-- Attachment #1: Type: text/plain, Size: 466 bytes --]

Hello,

It's been a little over 3 months since the GDB 7.5 release, and
looking at the ChangeLog on the branch since then, it looks like
it might be worth making a .1 release.

FYI: Attached are the list of changes that were documented in
the various ChangeLog files since the branch was created (changes
to the testsuite directory excised - for a shorter list).

Given that the US is probably busy eating turkey, I propose we
wait a bit, so Wed Nov 28th.

-- 
Joel

[-- Attachment #2: gdb-7.5-changes.diff --]
[-- Type: text/x-diff, Size: 10971 bytes --]

diff --git a/gdb/ChangeLog b/gdb/ChangeLog
index 45de328..d1d4f78 100644
--- a/gdb/ChangeLog
+++ b/gdb/ChangeLog
@@ -1,3 +1,243 @@
+2012-11-15  Luis Machado  <lgustavo@codesourcery.com>
+
+	* value.c (value_actual_type): Check for TYPE_CODE_STRUCT
+	target types.
+
+2012-11-10  H.J. Lu  <hongjiu.lu@intel.com>
+
+	PR backtrace/14646
+	PR gdb/14647
+	* i386-tdep.h (gdbarch_tdep): Remove sp_regnum_from_eax and
+	pc_regnum_from_eax.
+	* i386-tdep.c (i386_gdbarch_init): Don't use sp_regnum_from_eax
+	nor pc_regnum_from_eax.
+	* amd64-tdep.c (amd64_x32_init_abi): Don't set sp_regnum_from_eax
+	nor pc_regnum_from_eax.
+
+2012-10-11  Doug Evans  <dje@google.com>
+
+	PR breakpoints/14643.
+	* linespec.c (struct ls_parser): New member keyword_ok.
+	(linespec_lexer_lex_string): Add comment.
+	(linespec_lexer_lex_one): Ignore keywords if it's the wrong place
+	for one.
+	(parse_linespec): Set keyword_ok.
+
+2012-10-06  Jan Kratochvil  <jan.kratochvil@redhat.com>
+
+	Fix crash during stepping on ppc32.
+	* ppc-linux-tdep.c (powerpc_linux_in_dynsym_resolve_code): Test NULL
+	SYM.
+
+2012-09-17  Jan Kratochvil  <jan.kratochvil@redhat.com>
+
+	* common/linux-ptrace.c: Change __i386__ to __i386__ || __x86_64__.
+	(linux_ptrace_test_ret_to_nx): Extend comment for x86_64.  Change
+	__i386__ to __i386__ || __x86_64__. Extend code also for __x86_64__.
+	Extend code also for PaX support.  Convert all gdb_assert to warning
+	calls.
+
+2012-09-17  Jan Kratochvil  <jan.kratochvil@redhat.com>
+
+	Implement auto-load user conveniences suggested by Doug Evans.
+	* auto-load.c: Include top.h.
+	(file_is_auto_load_safe): New variable advice_printed.  Print advice.
+	(_initialize_auto_load): New variable scripts_directory_help.  Mention
+	GDBPY_AUTO_FILE_NAME and GDB_AUTO_FILE_NAME for set auto-load
+	scripts-directory.  Document in online help one can use also files for
+	set auto-load safe-path.
+	* python/py-auto-load.c: (GDBPY_AUTO_FILE_NAME): Move it from here ...
+	* python/python.h (GDBPY_AUTO_FILE_NAME): ... to here.
+
+2012-09-17  Jan Kratochvil  <jan.kratochvil@redhat.com>
+
+	PR 14119
+	* frame.c (skip_inlined_frames): Skip also TAILCALL_FRAME frames.
+	(frame_pop): Drop also TAILCALL_FRAME frames.
+	* infcmd.c (finish_command): Ignore also TAILCALL_FRAME frames.
+
+2012-09-17  Jan Kratochvil  <jan.kratochvil@redhat.com>
+	    Pedro Alves  <palves@redhat.com>
+
+	PR 14548
+	* infrun.c (handle_inferior_event): Do not reverse-continue back to the
+	function start if we are already at function start.  Both for
+	reverse-next and for reverse-step into function without line number
+	info.
+
+2012-08-29  Doug Evans  <dje@google.com>
+
+	* main.c (print_gdb_help): Remove reference to
+	--use-deprecated-index-sections.
+
+2012-08-27  Eli Zaretskii  <eliz@gnu.org>
+	    Jan Kratochvil  <jan.kratochvil@redhat.com>
+
+	* auto-load.c (auto_load_objfile_script): Rename to ...
+	(auto_load_objfile_script_1): ... here, change variable realname to
+	parameter realname, document it, add return value, add variable retval.
+	(auto_load_objfile_script): New function.
+
+2012-08-27  Jan Kratochvil  <jan.kratochvil@redhat.com>
+
+	* cli/cli-decode.c (print_doc_line): Keep skipping '.' and ',' not
+	followed by a whitespace.
+
+2012-08-27  Jan Kratochvil  <jan.kratochvil@redhat.com>
+
+	PR gdb/14494.
+	* dwarf2read.c (dwarf2_locate_sections): Move variable aflag here.
+	Move the SEC_HAS_CONTENTS check here - for any NAMES use.
+	(dwarf2_locate_sections) <eh_frame>: Move the variable and check from
+	here.
+
+2012-08-22  Keith Seitz  <keiths@redhat.com>
+
+	* defs.h: Include build-gnulib/config.h
+
+2012-08-21  Pierre Muller  <muller@ics.u-strasbg.fr>
+
+	* symfile.c (allocate_symtab): Use host_address_to_string
+	function instead of cast of pointer to long which is not
+	compatible with x86_64-w64-mingw32 build.
+
+2012-08-17  Joel Brobecker  <brobecker@adacore.com>
+
+	* version.in: Set version to 7.5.0.20120817-cvs.
+
+2012-08-17  Joel Brobecker  <brobecker@adacore.com>
+
+	GDB 7.5 released.
+
+2012-08-17  Joel Brobecker  <brobecker@adacore.com>
+
+	* version.in: Update GDB version number to 7.5.
+
+2012-08-16  Tom Tromey  <tromey@redhat.com>
+
+	PR python/14386:
+	* varobj.c (update_dynamic_varobj_children): Don't call
+	PyIter_Check.
+
+2012-08-06  Tom Tromey  <tromey@redhat.com>
+
+	* dwarf2-frame.c (clear_pointer_cleanup): New function.
+	(dwarf2_frame_cache): Use it.
+	* frame-unwind.h (frame_sniffer_ftype): Document prologue
+	cache initialization constraint.
+
+2012-08-06  Tom Tromey  <tromey@redhat.com>
+
+	PR cli/14392:
+	* cli/cli-cmds.c (list_command): Filter 'sals_end'.
+
+2012-08-03  Tom Tromey  <tromey@redhat.com>
+
+	* dwarf2-frame.c (dwarf2_frame_cache): Call do_cleanups before
+	return.
+
+2012-08-02  Ulrich Weigand  <ulrich.weigand@linaro.org>
+
+	* inf-child.c (inf_child_fileio_pwrite): If pwrite fails, fall back
+	to attempting lseek/write.
+	(inf_child_fileio_pread): Likewise for pread.
+
+2012-08-01  Richard Henderson  <rth@redhat.com>
+
+	* configure.tgt (m32c-*-*) [gdb_target_obs]: Remove prologue-value.o.
+	(mep-*-*) [gdb_target_obs]: Likewise.
+
+2012-08-01  Ulrich Weigand  <ulrich.weigand@linaro.org>
+
+	* arm-linux-nat.c (arm_linux_hw_breakpoint_initialize): Do not
+	attempt to 4-byte-align HW breakpoint addresses for Thumb.
+
+2012-07-31  Andreas Schwab  <schwab@linux-m68k.org>
+
+	* ppc-linux-tdep.c (ppc_linux_init_abi): Register
+	linux_get_siginfo_type.
+
+2012-07-31  Jan Kratochvil  <jan.kratochvil@redhat.com>
+
+	* infcall.c (call_function_by_hand): Move BP_ADDR comment to
+	AT_ENTRY_POINT.
+	(call_function_by_hand) <ON_STACK>: Call write_memory with
+	gdbarch_breakpoint_from_pc, if possible.
+	(call_function_by_hand) <AT_ENTRY_POINT>: The BP_ADDR comment is moved
+	here.
+
+2012-07-30  Keith Seitz  <keiths@redhat.com>
+
+	* linespec.c (linespec_lexer_lex_number): A number followed
+	by quotes is a valid number, too.
+
+2012-07-26  Keith Seitz  <keiths@redhat.com>
+
+	* linespec.c (linespec_lexer_lex_number): The input
+	is also a valid number if the next character is a comma
+	or colon.
+
+2012-07-25  Ralf Corsépius  <ralf.corsepius@rtems.org>
+
+	* configure.tgt: Add v850*-*-rtems*.
+
+2012-07-25  Hui Zhu  <hui_zhu@mentor.com>
+
+	* linespec.c (linespec_lexer_lex_number): Update comments,
+	change the return and add check to make sure the input is
+	the decimal numbers.
+	(linespec_lexer_lex_one): If linespec_lexer_lex_number return
+	false, call linespec_lexer_lex_string.
+
+2012-07-24  Marc Khouzam  <marc.khouzam@ericsson.com>
+
+	* breakpoint.c (create_breakpoint): Store condition for pending
+	breakpoints.
+
+2012-07-23  Keith Seitz  <keiths@redhat.com>
+
+	* linespec.c (convert_linespec_to_sal): Don't add
+	any symbols to the result vector if symbol_to_sal
+	returns zero.
+
+2012-07-20  Doug Evans  <dje@google.com>
+
+	* NEWS: Document new options "set/show use-deprecated-index-sections",
+	and delete reference to --use-deprecated-index-sections.
+	* symfile.h (use_deprecated_index_sections): Delete.
+	* dwarf2read.c (use_deprecated_index_sections): Make static.
+	(read_index_from_section): Update wording of how to load
+	deprecated index sections.
+	(_initialize_dwarf2_read): New options
+	"set/show use-deprecated-index-sections".
+	* main.c (captured_main): Delete --use-deprecated-index-sections.
+
+2012-07-18  Joel Brobecker  <brobecker@adacore.com>
+
+	* version.in: Set version to 7.4.91.20120718-cvs.
+
+2012-07-18  Joel Brobecker  <brobecker@adacore.com>
+
+	GDB 7.4.91 released.
+
+2012-07-18  Joel Brobecker  <brobecker@adacore.com>
+
+	* NEWS: Replace "Changes since GDB 7.4" with "Changes in GDB 7.5".
+
+2012-07-18  Joel Brobecker  <brobecker@adacore.com>
+
+	* version.in: Update GDB version number to 7.4.91.
+
+2012-07-18  Joel Brobecker  <brobecker@adacore.com>
+
+	* configure.ac (development): Set to false.
+	* configure: Regenerate.
+
+2012-07-18  Joel Brobecker  <brobecker@adacore.com>
+
+	GDB 7.5 branch created (branch timestamp: 2012-07-18 03:22 UTC)
+	* version.in: Bump version to 7.4.90.20120718-cvs.
+
 2012-07-16  Jan Kratochvil  <jan.kratochvil@redhat.com>
 
 	PR 11914
diff --git a/gdb/doc/ChangeLog b/gdb/doc/ChangeLog
index 97b9a17..762c1dc 100644
--- a/gdb/doc/ChangeLog
+++ b/gdb/doc/ChangeLog
@@ -1,3 +1,21 @@
+2012-08-27  Jan Kratochvil  <jan.kratochvil@redhat.com>
+
+	* gdb.texinfo (objfile-gdb.py file): New paragraph for .exe stripping.
+
+2012-08-07  Jan Kratochvil  <jan.kratochvil@redhat.com>
+
+	* gdbint.texinfo (Debugging GDB): In section
+	'Debugging @value{GDBN} with itself' change .gdbinit for gdb-gdb.gdb.
+	Mention also gdb-gdb.py.
+
+2012-07-20  Doug Evans  <dje@google.com>
+
+	* gdb.texinfo (Mode Options): Delete --use-deprecated-index-sections.
+	(Index Files): Document how to control the use of deprecated index
+	sections.
+	(Index Section Format): Replace --use-deprecated-index-sections with
+	"set use-deprecated-index-sections on".
+
 2012-07-12  Eli Zaretskii  <eliz@gnu.org>
 
 	* gdbint.texinfo: Remove @syncodeindex directives that put all the
diff --git a/gdb/gdbserver/ChangeLog b/gdb/gdbserver/ChangeLog
index 3757add..f741bd6 100644
--- a/gdb/gdbserver/ChangeLog
+++ b/gdb/gdbserver/ChangeLog
@@ -1,3 +1,33 @@
+2012-09-05  Michael Brandt  <michael.brandt@axis.com>
+
+	* linux-crisv32-low.c: Fix compile errors.
+
+2012-08-22  Keith Seitz  <keiths@redhat.com>
+
+	* server.h: Include build-gnulib-gbserver/config.h.
+	* gdbreplay.c: Likewise.
+
+2012-08-02  Ulrich Weigand  <ulrich.weigand@linaro.org>
+
+	* hostio.c (handle_pread): If pread fails, fall back to attempting
+	lseek/read.
+	(handle_pwrite): Likewise for pwrite.
+
+2012-08-01  Ulrich Weigand  <ulrich.weigand@linaro.org>
+
+	* linux-arm-low.c (arm_linux_hw_point_initialize): Distinguish
+	between unsupported TYPE and unimplementable ADDR/LEN combination.
+	(arm_insert_point): Act on new return value.
+
+2012-08-01  Ulrich Weigand  <ulrich.weigand@linaro.org>
+
+	* linux-arm-low.c (arm_linux_hw_point_initialize): Do not attempt
+	to 4-byte-align HW breakpoint addresses for Thumb.
+
+2012-07-27  Yao Qi  <yao@codesourcery.com>
+
+	* server.c (process_point_options): Stop at 'X' when parsing.
+
 2012-07-07  Jan Kratochvil  <jan.kratochvil@redhat.com>
 
 	* gdbserver/linux-low.c (initialize_low): Call
diff --git a/sim/bfin/ChangeLog b/sim/bfin/ChangeLog
index 5d09b5a..8283086 100644
--- a/sim/bfin/ChangeLog
+++ b/sim/bfin/ChangeLog
@@ -1,3 +1,16 @@
+2012-07-31  Mike Frysinger  <vapier@gentoo.org>
+
+	* machs.c (bf54x_roms): Pass 0x1000 to alias field of BFROM, and
+	0x10000 to the alias field of BFROMA.
+	(bf561_roms): Pass 0x1000 to alias field of BFROM.
+	(bf59x_roms): Pass 0x10000 to alias field of BFROMA.
+
+2012-07-31  Mike Frysinger  <vapier@gentoo.org>
+
+	* machs.c (bfin_reg_fetch): Change return 0 to return -1, and
+	return -1 to return 4.
+	(bfin_reg_store): Likewise.
+
 2012-06-15  Joel Brobecker  <brobecker@adacore.com>
 
 	* config.in, configure: Regenerate.

Re: New GDB 7.5.x release sometime next week?

[Mike Frysinger]

[-- Attachment #1: Type: Text/Plain, Size: 663 bytes --]

On Thursday 22 November 2012 11:31:46 Joel Brobecker wrote:
> It's been a little over 3 months since the GDB 7.5 release, and
> looking at the ChangeLog on the branch since then, it looks like
> it might be worth making a .1 release.
> 
> FYI: Attached are the list of changes that were documented in
> the various ChangeLog files since the branch was created (changes
> to the testsuite directory excised - for a shorter list).

how about this in libiberty/:
2012-07-31  Mike Frysinger  <vapier@gentoo.org>

	* md5.c (md5_finish_ctx): Declare swap_bytes.  Assign SWAP() output
	to swap_bytes, and then call memcpy to move it to ctx->buffer.
-mike

[-- Attachment #2: This is a digitally signed message part. --]
[-- Type: application/pgp-signature, Size: 836 bytes --]

Re: New GDB 7.5.x release sometime next week?

[Joel Brobecker]

> how about this in libiberty/:
> 2012-07-31  Mike Frysinger  <vapier@gentoo.org>
> 
> 	* md5.c (md5_finish_ctx): Declare swap_bytes.  Assign SWAP() output
> 	to swap_bytes, and then call memcpy to move it to ctx->buffer.

As always, a quick explanation of what it fixes, and why it is
important enough that it should be included in the release would help.
Just from the ChangeLog entry alone, I am having a difficult time
evaluating it.

Thanks!
-- 
Joel

Re: New GDB 7.5.x release sometime next week?

[Mike Frysinger]

[-- Attachment #1: Type: Text/Plain, Size: 1267 bytes --]

On Tuesday 27 November 2012 15:29:13 Joel Brobecker wrote:
> > how about this in libiberty/:
> > 2012-07-31  Mike Frysinger  <vapier@gentoo.org>
> > 
> > 	* md5.c (md5_finish_ctx): Declare swap_bytes.  Assign SWAP() output
> > 	to swap_bytes, and then call memcpy to move it to ctx->buffer.
> 
> As always, a quick explanation of what it fixes, and why it is
> important enough that it should be included in the release would help.
> Just from the ChangeLog entry alone, I am having a difficult time
> evaluating it.

Subject: [PATCH] libiberty/md5: fix strict alias warnings

Current libiberty md5 code triggers these warnings with gcc-4.7.1 for me:

libiberty/md5.c: In function 'md5_finish_ctx':
libiberty/md5.c:117:3: warning: dereferencing type-punned pointer will break 
strict-aliasing rules [-Wstrict-aliasing]
libiberty/md5.c:118:3: warning: dereferencing type-punned pointer will break 
strict-aliasing rules [-Wstrict-aliasing]

The change below fixes things for me.  The optimized output (-O2) is the same
before/after my change on x86_64-linux.  I imagine it'll be the same for most
targets.  It seems simpler than using a union on the md5_ctx buffer since these
are the only two locations in the code where this occurs.
-mike

[-- Attachment #2: This is a digitally signed message part. --]
[-- Type: application/pgp-signature, Size: 836 bytes --]

Re: New GDB 7.5.x release sometime next week?

[Joel Brobecker]

> Subject: [PATCH] libiberty/md5: fix strict alias warnings
> 
> Current libiberty md5 code triggers these warnings with gcc-4.7.1 for me:
> 
> libiberty/md5.c: In function 'md5_finish_ctx':
> libiberty/md5.c:117:3: warning: dereferencing type-punned pointer will break 
> strict-aliasing rules [-Wstrict-aliasing]
> libiberty/md5.c:118:3: warning: dereferencing type-punned pointer will break 
> strict-aliasing rules [-Wstrict-aliasing]
> 
> The change below fixes things for me.  The optimized output (-O2) is
> the same before/after my change on x86_64-linux.  I imagine it'll be
> the same for most targets.  It seems simpler than using a union on the
> md5_ctx buffer since these are the only two locations in the code
> where this occurs.

I am on the fence regarding this change. On the one hand, it seems
hardly critical, since it only fixes some warnings, and does not seem
to cause any improvement in the actual code generated. On the other
hand, this module appears to be completely unused in all of GDB (?).
So the risk would be small. For that reason, and the fact that the code
has been in since end of July, I think it's OK to include it, and
avoid the warnings.

Thanks,
-- 
Joel

Re: New GDB 7.5.x release sometime next week?

[Mike Frysinger]

[-- Attachment #1: Type: Text/Plain, Size: 1586 bytes --]

On Tuesday 27 November 2012 16:24:46 Joel Brobecker wrote:
> > Subject: [PATCH] libiberty/md5: fix strict alias warnings
> > 
> > Current libiberty md5 code triggers these warnings with gcc-4.7.1 for me:
> > 
> > libiberty/md5.c: In function 'md5_finish_ctx':
> > libiberty/md5.c:117:3: warning: dereferencing type-punned pointer will
> > break strict-aliasing rules [-Wstrict-aliasing]
> > libiberty/md5.c:118:3: warning: dereferencing type-punned pointer will
> > break strict-aliasing rules [-Wstrict-aliasing]
> > 
> > The change below fixes things for me.  The optimized output (-O2) is
> > the same before/after my change on x86_64-linux.  I imagine it'll be
> > the same for most targets.  It seems simpler than using a union on the
> > md5_ctx buffer since these are the only two locations in the code
> > where this occurs.
> 
> I am on the fence regarding this change. On the one hand, it seems
> hardly critical, since it only fixes some warnings, and does not seem
> to cause any improvement in the actual code generated. On the other
> hand, this module appears to be completely unused in all of GDB (?).
> So the risk would be small. For that reason, and the fact that the code
> has been in since end of July, I think it's OK to include it, and
> avoid the warnings.

yeah, i don't think it's critical.  i mentioned it because i was carrying it 
in Gentoo's 7.5 patchset (due to the build warnings).

i also can't find any md5 usage in gdb.  binutils uses it in the linker, but 
that seems to be all.

i'll commit it to the branch.
-mike

[-- Attachment #2: This is a digitally signed message part. --]
[-- Type: application/pgp-signature, Size: 836 bytes --]