Mirror of the gdb-patches mailing list
 help / color / mirror / Atom feed
* Breakage on builder Fedora-i686, revision 4ee89e903de920496a69c01df70db13a70a2a0be
  2018-03-20 14:14 Oh dear. I regret to inform you that commit 4ee89e903de920496a69c01df70db13a70a2a0be might be unfortunate gdb-buildbot
@ 2018-03-20 14:14 ` gdb-buildbot
  2018-03-20 14:16 ` Breakage on builder Fedora-x86_64-native-gdbserver-m32, " gdb-buildbot
                   ` (8 subsequent siblings)
  9 siblings, 0 replies; 11+ messages in thread
From: gdb-buildbot @ 2018-03-20 14:14 UTC (permalink / raw)
  To: Stephen Roberts, gdb-patches

Unfortunately it seems that there is a breakage on GDB.

Commit title: 'Replace the linear search in find_pc_sect_line with a binary search.'
Revision: 4ee89e903de920496a69c01df70db13a70a2a0be

You can find more details below:

+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+

Buildslave:
	fedora-x86-64-2

Full Build URL:
	<http://gdb-build.sergiodj.net/builders/Fedora-i686/builds/9119>

Commit(s) tested:
	4ee89e903de920496a69c01df70db13a70a2a0be

Author(s) (in the same order as the commits):
	Stephen Roberts <stephen.roberts@arm.com>

Subject:
	Replace the linear search in find_pc_sect_line with a binary search.

Testsuite log (gdb.sum and gdb.log) URL(s):
	<http://gdb-build.sergiodj.net/results/Fedora-i686/4e/4ee89e903de920496a69c01df70db13a70a2a0be/>

*** Failed to compiled GDB.  ***
============================

+++ The full log is too big to be posted here.
+++ These are the last 100 lines of it.

  CXX    gdbarch.o
  CXX    gdbarch-selftests.o
  CXX    gdbtypes.o
  CXX    gnu-v2-abi.o
  CXX    gnu-v3-abi.o
  CXX    go-lang.o
  CXX    go-typeprint.o
  CXX    go-valprint.o
  CXX    inf-child.o
  CXX    inf-loop.o
  CXX    infcall.o
  CXX    infcmd.o
  CXX    inferior.o
  CXX    infrun.o
  CXX    inline-frame.o
  CXX    interps.o
  CXX    jit.o
  CXX    language.o
  CXX    linespec.o
  CXX    location.o
  CXX    m2-lang.o
  CXX    m2-typeprint.o
  CXX    m2-valprint.o
  CXX    macrocmd.o
  CXX    macroexp.o
  CXX    macroscope.o
  CXX    macrotab.o
  CXX    main.o
  CXX    maint.o
  CXX    mdebugread.o
  CXX    mem-break.o
  CXX    memattr.o
  CXX    memory-map.o
  CXX    memrange.o
  CXX    minidebug.o
  CXX    minsyms.o
  CXX    mipsread.o
  CXX    namespace.o
  CXX    objc-lang.o
  CXX    objfiles.o
  CXX    observable.o
  CXX    opencl-lang.o
  CXX    osabi.o
  CXX    osdata.o
  CXX    p-lang.o
  CXX    p-typeprint.o
  CXX    p-valprint.o
  CXX    parse.o
  CXX    printcmd.o
  CXX    probe.o
  CXX    producer.o
  CXX    progspace.o
  CXX    progspace-and-thread.o
  CXX    prologue-value.o
  CXX    psymtab.o
  CXX    record.o
  CXX    record-btrace.o
  CXX    record-full.o
  CXX    regcache.o
  CXX    regcache-dump.o
  CXX    reggroups.o
  CXX    registry.o
  CXX    remote.o
  CXX    remote-fileio.o
  CXX    remote-notif.o
  CXX    reverse.o
  CXX    rust-lang.o
  CXX    sentinel-frame.o
  CXX    ser-event.o
  CXX    serial.o
  CXX    skip.o
  CXX    solib.o
  CXX    solib-target.o
  CXX    source.o
  CXX    stabsread.o
  CXX    stack.o
  CXX    std-regs.o
  CXX    symfile.o
  CXX    symfile-debug.o
  CXX    symmisc.o
  CXX    symtab.o
  CXX    target.o
  CXX    target-dcache.o
../../binutils-gdb/gdb/symtab.c: In function symtab_and_line find_pc_sect_line(CORE_ADDR, obj_section*, int):
../../binutils-gdb/gdb/symtab.c:3224:5: error: this if clause does not guard... [-Werror=misleading-indentation]
     if (item != first)
     ^~
../../binutils-gdb/gdb/symtab.c:3237:7: note: ...this statement, but the latter is misleadingly indented as if it were guarded by the if
       if (prev && prev->line && (!best || prev->pc > best->pc))
       ^~
  CXX    target-descriptions.o
  CXX    target-memory.o
cc1plus: all warnings being treated as errors
make[2]: *** [Makefile:1607: symtab.o] Error 1
make[2]: *** Waiting for unfinished jobs....
make[2]: Leaving directory '/home/gdb-buildbot-2/fedora-x86-64-2/fedora-i686/build/gdb'
make[1]: *** [Makefile:8793: all-gdb] Error 2
make: *** [Makefile:850: all] Error 2
make[1]: Leaving directory '/home/gdb-buildbot-2/fedora-x86-64-2/fedora-i686/build'
============================


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

* Oh dear. I regret to inform you that commit 4ee89e903de920496a69c01df70db13a70a2a0be might be unfortunate
@ 2018-03-20 14:14 gdb-buildbot
  2018-03-20 14:14 ` Breakage on builder Fedora-i686, revision 4ee89e903de920496a69c01df70db13a70a2a0be gdb-buildbot
                   ` (9 more replies)
  0 siblings, 10 replies; 11+ messages in thread
From: gdb-buildbot @ 2018-03-20 14:14 UTC (permalink / raw)
  To: Stephen Roberts, gdb-patches

My lords, ladies, gentlemen, members of the public.

It is a matter of great regret and sadness to inform you that commit:

	Replace the linear search in find_pc_sect_line with a binary search.
	4ee89e903de920496a69c01df70db13a70a2a0be

might have made GDB unwell.  Since I am just your Butler BuildBot,
I kindly ask that a human superior officer double-check this.

Please note that if you are reading this message on gdb-patches, there might
be other builders broken.

You can find more details about the unfortunate breakage in the next messages.

Cheers,

Your GDB BuildBot.


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

* Breakage on builder Fedora-x86_64-m64, revision 4ee89e903de920496a69c01df70db13a70a2a0be
  2018-03-20 14:14 Oh dear. I regret to inform you that commit 4ee89e903de920496a69c01df70db13a70a2a0be might be unfortunate gdb-buildbot
  2018-03-20 14:14 ` Breakage on builder Fedora-i686, revision 4ee89e903de920496a69c01df70db13a70a2a0be gdb-buildbot
  2018-03-20 14:16 ` Breakage on builder Fedora-x86_64-native-gdbserver-m32, " gdb-buildbot
@ 2018-03-20 14:16 ` gdb-buildbot
  2018-03-20 14:17 ` Breakage on builder Fedora-s390x-m64, " gdb-buildbot
                   ` (6 subsequent siblings)
  9 siblings, 0 replies; 11+ messages in thread
From: gdb-buildbot @ 2018-03-20 14:16 UTC (permalink / raw)
  To: Stephen Roberts, gdb-patches

Unfortunately it seems that there is a breakage on GDB.

Commit title: 'Replace the linear search in find_pc_sect_line with a binary search.'
Revision: 4ee89e903de920496a69c01df70db13a70a2a0be

You can find more details below:

+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+

Buildslave:
	fedora-x86-64-1

Full Build URL:
	<http://gdb-build.sergiodj.net/builders/Fedora-x86_64-m64/builds/9137>

Commit(s) tested:
	4ee89e903de920496a69c01df70db13a70a2a0be

Author(s) (in the same order as the commits):
	Stephen Roberts <stephen.roberts@arm.com>

Subject:
	Replace the linear search in find_pc_sect_line with a binary search.

Testsuite log (gdb.sum and gdb.log) URL(s):
	<http://gdb-build.sergiodj.net/results/Fedora-x86_64-m64/4e/4ee89e903de920496a69c01df70db13a70a2a0be/>

*** Failed to compiled GDB.  ***
============================

+++ The full log is too big to be posted here.
+++ These are the last 100 lines of it.

                 from ../../binutils-gdb/gdb/infrun.c:26:
../../binutils-gdb/gdb/progspace.h:284:31: warning: *((void*)(& maybe_restore_inferior)+32).scoped_restore_current_program_space::m_saved_pspace may be used uninitialized in this function [-Wmaybe-uninitialized]
   { set_current_program_space (m_saved_pspace); }
     ~~~~~~~~~~~~~~~~~~~~~~~~~~^~~~~~~~~~~~~~~~
../../binutils-gdb/gdb/infrun.c:927:6: note: *((void*)(& maybe_restore_inferior)+32).scoped_restore_current_program_space::m_saved_pspace was declared here
      maybe_restore_inferior;
      ^~~~~~~~~~~~~~~~~~~~~~
  CXX    linespec.o
  CXX    location.o
  CXX    m2-lang.o
  CXX    m2-typeprint.o
  CXX    m2-valprint.o
  CXX    macrocmd.o
  CXX    macroexp.o
  CXX    macroscope.o
  CXX    macrotab.o
  CXX    main.o
  CXX    maint.o
  CXX    mdebugread.o
  CXX    mem-break.o
  CXX    memattr.o
  CXX    memory-map.o
  CXX    memrange.o
  CXX    minidebug.o
  CXX    minsyms.o
  CXX    mipsread.o
  CXX    namespace.o
  CXX    objc-lang.o
  CXX    objfiles.o
  CXX    observable.o
  CXX    opencl-lang.o
  CXX    osabi.o
  CXX    osdata.o
  CXX    p-lang.o
  CXX    p-typeprint.o
  CXX    p-valprint.o
  CXX    parse.o
  CXX    printcmd.o
  CXX    probe.o
  CXX    producer.o
  CXX    progspace.o
  CXX    progspace-and-thread.o
  CXX    prologue-value.o
  CXX    psymtab.o
  CXX    record.o
  CXX    record-btrace.o
  CXX    record-full.o
In file included from ../../binutils-gdb/gdb/exceptions.h:23:0,
                 from ../../binutils-gdb/gdb/utils.h:24,
                 from ../../binutils-gdb/gdb/defs.h:676,
                 from ../../binutils-gdb/gdb/record-btrace.c:22:
../../binutils-gdb/gdb/ui-out.h: In function void btrace_insn_history(ui_out*, const btrace_thread_info*, const btrace_insn_iterator*, const btrace_insn_iterator*, gdb_disassembly_flags):
../../binutils-gdb/gdb/ui-out.h:197:18: warning: asm_list.ui_out_emit_type<(ui_out_type)1>::m_uiout may be used uninitialized in this function [-Wmaybe-uninitialized]
     m_uiout->end (Type);
     ~~~~~~~~~~~~~^~~~~~
../../binutils-gdb/gdb/record-btrace.c:658:35: note: asm_list.ui_out_emit_type<(ui_out_type)1>::m_uiout was declared here
   gdb::optional<ui_out_emit_list> asm_list;
                                   ^~~~~~~~
  CXX    regcache.o
  CXX    regcache-dump.o
  CXX    reggroups.o
  CXX    registry.o
  CXX    remote.o
  CXX    remote-fileio.o
  CXX    remote-notif.o
  CXX    reverse.o
  CXX    rust-lang.o
  CXX    sentinel-frame.o
  CXX    ser-event.o
  CXX    serial.o
  CXX    skip.o
  CXX    solib.o
  CXX    solib-target.o
  CXX    source.o
  CXX    stabsread.o
  CXX    stack.o
  CXX    std-regs.o
  CXX    symfile.o
  CXX    symfile-debug.o
  CXX    symmisc.o
  CXX    symtab.o
  CXX    target.o
  CXX    target-dcache.o
../../binutils-gdb/gdb/symtab.c: In function symtab_and_line find_pc_sect_line(CORE_ADDR, obj_section*, int):
../../binutils-gdb/gdb/symtab.c:3224:5: error: this if clause does not guard... [-Werror=misleading-indentation]
     if (item != first)
     ^~
../../binutils-gdb/gdb/symtab.c:3237:7: note: ...this statement, but the latter is misleadingly indented as if it were guarded by the if
       if (prev && prev->line && (!best || prev->pc > best->pc))
       ^~
  CXX    target-descriptions.o
  CXX    target-memory.o
cc1plus: all warnings being treated as errors
make[2]: *** [Makefile:1607: symtab.o] Error 1
make[2]: *** Waiting for unfinished jobs....
make[2]: Leaving directory '/home/gdb-buildbot/fedora-x86-64-1/fedora-x86-64/build/gdb'
make[1]: *** [Makefile:8793: all-gdb] Error 2
make: *** [Makefile:850: all] Error 2
make[1]: Leaving directory '/home/gdb-buildbot/fedora-x86-64-1/fedora-x86-64/build'
============================


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

* Breakage on builder Fedora-x86_64-native-gdbserver-m32, revision 4ee89e903de920496a69c01df70db13a70a2a0be
  2018-03-20 14:14 Oh dear. I regret to inform you that commit 4ee89e903de920496a69c01df70db13a70a2a0be might be unfortunate gdb-buildbot
  2018-03-20 14:14 ` Breakage on builder Fedora-i686, revision 4ee89e903de920496a69c01df70db13a70a2a0be gdb-buildbot
@ 2018-03-20 14:16 ` gdb-buildbot
  2018-03-20 14:16 ` Breakage on builder Fedora-x86_64-m64, " gdb-buildbot
                   ` (7 subsequent siblings)
  9 siblings, 0 replies; 11+ messages in thread
From: gdb-buildbot @ 2018-03-20 14:16 UTC (permalink / raw)
  To: Stephen Roberts, gdb-patches

Unfortunately it seems that there is a breakage on GDB.

Commit title: 'Replace the linear search in find_pc_sect_line with a binary search.'
Revision: 4ee89e903de920496a69c01df70db13a70a2a0be

You can find more details below:

+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+

Buildslave:
	fedora-x86-64-3

Full Build URL:
	<http://gdb-build.sergiodj.net/builders/Fedora-x86_64-native-gdbserver-m32/builds/9080>

Commit(s) tested:
	4ee89e903de920496a69c01df70db13a70a2a0be

Author(s) (in the same order as the commits):
	Stephen Roberts <stephen.roberts@arm.com>

Subject:
	Replace the linear search in find_pc_sect_line with a binary search.

Testsuite log (gdb.sum and gdb.log) URL(s):
	<http://gdb-build.sergiodj.net/results/Fedora-x86_64-native-gdbserver-m32/4e/4ee89e903de920496a69c01df70db13a70a2a0be/>

*** Failed to compiled GDB.  ***
============================

+++ The full log is too big to be posted here.
+++ These are the last 100 lines of it.

In file included from ../../binutils-gdb/gdb/inferior.h:48:0,
                 from ../../binutils-gdb/gdb/infrun.c:26:
../../binutils-gdb/gdb/progspace.h:284:31: warning: *((void*)(& maybe_restore_inferior)+32).scoped_restore_current_program_space::m_saved_pspace may be used uninitialized in this function [-Wmaybe-uninitialized]
   { set_current_program_space (m_saved_pspace); }
     ~~~~~~~~~~~~~~~~~~~~~~~~~~^~~~~~~~~~~~~~~~
../../binutils-gdb/gdb/infrun.c:927:6: note: *((void*)(& maybe_restore_inferior)+32).scoped_restore_current_program_space::m_saved_pspace was declared here
      maybe_restore_inferior;
      ^~~~~~~~~~~~~~~~~~~~~~
  CXX    location.o
  CXX    m2-lang.o
  CXX    m2-typeprint.o
  CXX    m2-valprint.o
  CXX    macrocmd.o
  CXX    macroexp.o
  CXX    macroscope.o
  CXX    macrotab.o
  CXX    main.o
  CXX    maint.o
  CXX    mdebugread.o
  CXX    mem-break.o
  CXX    memattr.o
  CXX    memory-map.o
  CXX    memrange.o
  CXX    minidebug.o
  CXX    minsyms.o
  CXX    mipsread.o
  CXX    namespace.o
  CXX    objc-lang.o
  CXX    objfiles.o
  CXX    observable.o
  CXX    opencl-lang.o
  CXX    osabi.o
  CXX    osdata.o
  CXX    p-lang.o
  CXX    p-typeprint.o
  CXX    p-valprint.o
  CXX    parse.o
  CXX    printcmd.o
  CXX    probe.o
  CXX    producer.o
  CXX    progspace.o
  CXX    progspace-and-thread.o
  CXX    prologue-value.o
  CXX    psymtab.o
  CXX    record.o
  CXX    record-btrace.o
  CXX    record-full.o
  CXX    regcache.o
In file included from ../../binutils-gdb/gdb/exceptions.h:23:0,
                 from ../../binutils-gdb/gdb/utils.h:24,
                 from ../../binutils-gdb/gdb/defs.h:676,
                 from ../../binutils-gdb/gdb/record-btrace.c:22:
../../binutils-gdb/gdb/ui-out.h: In function void btrace_insn_history(ui_out*, const btrace_thread_info*, const btrace_insn_iterator*, const btrace_insn_iterator*, gdb_disassembly_flags):
../../binutils-gdb/gdb/ui-out.h:197:18: warning: asm_list.ui_out_emit_type<(ui_out_type)1>::m_uiout may be used uninitialized in this function [-Wmaybe-uninitialized]
     m_uiout->end (Type);
     ~~~~~~~~~~~~~^~~~~~
../../binutils-gdb/gdb/record-btrace.c:658:35: note: asm_list.ui_out_emit_type<(ui_out_type)1>::m_uiout was declared here
   gdb::optional<ui_out_emit_list> asm_list;
                                   ^~~~~~~~
  CXX    regcache-dump.o
  CXX    reggroups.o
  CXX    registry.o
  CXX    remote.o
  CXX    remote-fileio.o
  CXX    remote-notif.o
  CXX    reverse.o
  CXX    rust-lang.o
  CXX    sentinel-frame.o
  CXX    ser-event.o
  CXX    serial.o
  CXX    skip.o
  CXX    solib.o
  CXX    solib-target.o
  CXX    source.o
  CXX    stabsread.o
  CXX    stack.o
  CXX    std-regs.o
  CXX    symfile.o
  CXX    symfile-debug.o
  CXX    symmisc.o
  CXX    symtab.o
  CXX    target.o
../../binutils-gdb/gdb/symtab.c: In function symtab_and_line find_pc_sect_line(CORE_ADDR, obj_section*, int):
../../binutils-gdb/gdb/symtab.c:3224:5: error: this if clause does not guard... [-Werror=misleading-indentation]
     if (item != first)
     ^~
../../binutils-gdb/gdb/symtab.c:3237:7: note: ...this statement, but the latter is misleadingly indented as if it were guarded by the if
       if (prev && prev->line && (!best || prev->pc > best->pc))
       ^~
  CXX    target-dcache.o
  CXX    target-descriptions.o
  CXX    target-memory.o
cc1plus: all warnings being treated as errors
make[2]: *** [Makefile:1607: symtab.o] Error 1
make[2]: *** Waiting for unfinished jobs....
make[2]: Leaving directory '/home/gdb-buildbot/fedora-x86-64-3/fedora-x86-64-native-gdbserver-m32/build/gdb'
make[1]: *** [Makefile:8793: all-gdb] Error 2
make: *** [Makefile:850: all] Error 2
make[1]: Leaving directory '/home/gdb-buildbot/fedora-x86-64-3/fedora-x86-64-native-gdbserver-m32/build'
============================


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

* Breakage on builder Fedora-s390x-m64, revision 4ee89e903de920496a69c01df70db13a70a2a0be
  2018-03-20 14:14 Oh dear. I regret to inform you that commit 4ee89e903de920496a69c01df70db13a70a2a0be might be unfortunate gdb-buildbot
                   ` (2 preceding siblings ...)
  2018-03-20 14:16 ` Breakage on builder Fedora-x86_64-m64, " gdb-buildbot
@ 2018-03-20 14:17 ` gdb-buildbot
  2018-03-20 14:17 ` Breakage on builder Fedora-x86_64-m32, " gdb-buildbot
                   ` (5 subsequent siblings)
  9 siblings, 0 replies; 11+ messages in thread
From: gdb-buildbot @ 2018-03-20 14:17 UTC (permalink / raw)
  To: Stephen Roberts, gdb-patches

Unfortunately it seems that there is a breakage on GDB.

Commit title: 'Replace the linear search in find_pc_sect_line with a binary search.'
Revision: 4ee89e903de920496a69c01df70db13a70a2a0be

You can find more details below:

+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+

Buildslave:
	marist-fedora-s390x

Full Build URL:
	<http://gdb-build.sergiodj.net/builders/Fedora-s390x-m64/builds/7982>

Commit(s) tested:
	4ee89e903de920496a69c01df70db13a70a2a0be

Author(s) (in the same order as the commits):
	Stephen Roberts <stephen.roberts@arm.com>

Subject:
	Replace the linear search in find_pc_sect_line with a binary search.

Testsuite log (gdb.sum and gdb.log) URL(s):
	<http://gdb-build.sergiodj.net/results/Fedora-s390x-m64/4e/4ee89e903de920496a69c01df70db13a70a2a0be/>

*** Failed to compiled GDB.  ***
============================

+++ The full log is too big to be posted here.
+++ These are the last 100 lines of it.

In file included from ../../binutils-gdb/gdb/inferior.h:48:0,
                 from ../../binutils-gdb/gdb/infrun.c:26:
../../binutils-gdb/gdb/progspace.h:284:31: warning: *((void*)(& maybe_restore_inferior)+32).scoped_restore_current_program_space::m_saved_pspace may be used uninitialized in this function [-Wmaybe-uninitialized]
   { set_current_program_space (m_saved_pspace); }
     ~~~~~~~~~~~~~~~~~~~~~~~~~~^~~~~~~~~~~~~~~~
../../binutils-gdb/gdb/infrun.c:927:6: note: *((void*)(& maybe_restore_inferior)+32).scoped_restore_current_program_space::m_saved_pspace was declared here
      maybe_restore_inferior;
      ^~~~~~~~~~~~~~~~~~~~~~
  CXX    jit.o
  CXX    language.o
  CXX    linespec.o
  CXX    location.o
  CXX    m2-lang.o
  CXX    m2-typeprint.o
  CXX    m2-valprint.o
  CXX    macrocmd.o
  CXX    macroexp.o
  CXX    macroscope.o
  CXX    macrotab.o
  CXX    main.o
  CXX    maint.o
  CXX    mdebugread.o
  CXX    mem-break.o
  CXX    memattr.o
  CXX    memory-map.o
  CXX    memrange.o
  CXX    minidebug.o
  CXX    minsyms.o
  CXX    mipsread.o
  CXX    namespace.o
  CXX    objc-lang.o
  CXX    objfiles.o
  CXX    observable.o
  CXX    opencl-lang.o
  CXX    osabi.o
  CXX    osdata.o
  CXX    p-lang.o
  CXX    p-typeprint.o
  CXX    p-valprint.o
  CXX    parse.o
  CXX    printcmd.o
  CXX    probe.o
  CXX    producer.o
  CXX    progspace.o
  CXX    progspace-and-thread.o
  CXX    prologue-value.o
  CXX    psymtab.o
  CXX    record.o
  CXX    record-btrace.o
  CXX    record-full.o
In file included from ../../binutils-gdb/gdb/exceptions.h:23:0,
                 from ../../binutils-gdb/gdb/utils.h:24,
                 from ../../binutils-gdb/gdb/defs.h:676,
                 from ../../binutils-gdb/gdb/record-btrace.c:22:
../../binutils-gdb/gdb/ui-out.h: In function void btrace_insn_history(ui_out*, const btrace_thread_info*, const btrace_insn_iterator*, const btrace_insn_iterator*, gdb_disassembly_flags):
../../binutils-gdb/gdb/ui-out.h:197:18: warning: asm_list.ui_out_emit_type<(ui_out_type)1>::m_uiout may be used uninitialized in this function [-Wmaybe-uninitialized]
     m_uiout->end (Type);
     ~~~~~~~~~~~~~^~~~~~
../../binutils-gdb/gdb/record-btrace.c:658:35: note: asm_list.ui_out_emit_type<(ui_out_type)1>::m_uiout was declared here
   gdb::optional<ui_out_emit_list> asm_list;
                                   ^~~~~~~~
  CXX    regcache.o
  CXX    regcache-dump.o
  CXX    reggroups.o
  CXX    registry.o
  CXX    remote.o
  CXX    remote-fileio.o
  CXX    remote-notif.o
  CXX    reverse.o
  CXX    rust-lang.o
  CXX    sentinel-frame.o
  CXX    ser-event.o
  CXX    serial.o
  CXX    skip.o
  CXX    solib.o
  CXX    solib-target.o
  CXX    source.o
  CXX    stabsread.o
  CXX    stack.o
  CXX    std-regs.o
  CXX    symfile.o
  CXX    symfile-debug.o
  CXX    symmisc.o
  CXX    symtab.o
  CXX    target.o
../../binutils-gdb/gdb/symtab.c: In function symtab_and_line find_pc_sect_line(CORE_ADDR, obj_section*, int):
../../binutils-gdb/gdb/symtab.c:3224:5: error: this if clause does not guard... [-Werror=misleading-indentation]
     if (item != first)
     ^~
../../binutils-gdb/gdb/symtab.c:3237:7: note: ...this statement, but the latter is misleadingly indented as if it were guarded by the if
       if (prev && prev->line && (!best || prev->pc > best->pc))
       ^~
cc1plus: all warnings being treated as errors
make[2]: *** [Makefile:1607: symtab.o] Error 1
make[2]: *** Waiting for unfinished jobs....
make[2]: Leaving directory '/home/mjw/gdb/build/fedora-s390x/build/gdb'
make[1]: *** [Makefile:8793: all-gdb] Error 2
make[1]: Leaving directory '/home/mjw/gdb/build/fedora-s390x/build'
make: *** [Makefile:850: all] Error 2
============================


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

* Breakage on builder Fedora-x86_64-m32, revision 4ee89e903de920496a69c01df70db13a70a2a0be
  2018-03-20 14:14 Oh dear. I regret to inform you that commit 4ee89e903de920496a69c01df70db13a70a2a0be might be unfortunate gdb-buildbot
                   ` (3 preceding siblings ...)
  2018-03-20 14:17 ` Breakage on builder Fedora-s390x-m64, " gdb-buildbot
@ 2018-03-20 14:17 ` gdb-buildbot
  2018-03-20 14:21 ` Breakage on builder Fedora-x86_64-native-extended-gdbserver-m32, " gdb-buildbot
                   ` (4 subsequent siblings)
  9 siblings, 0 replies; 11+ messages in thread
From: gdb-buildbot @ 2018-03-20 14:17 UTC (permalink / raw)
  To: Stephen Roberts, gdb-patches

Unfortunately it seems that there is a breakage on GDB.

Commit title: 'Replace the linear search in find_pc_sect_line with a binary search.'
Revision: 4ee89e903de920496a69c01df70db13a70a2a0be

You can find more details below:

+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+

Buildslave:
	fedora-x86-64-4

Full Build URL:
	<http://gdb-build.sergiodj.net/builders/Fedora-x86_64-m32/builds/9095>

Commit(s) tested:
	4ee89e903de920496a69c01df70db13a70a2a0be

Author(s) (in the same order as the commits):
	Stephen Roberts <stephen.roberts@arm.com>

Subject:
	Replace the linear search in find_pc_sect_line with a binary search.

Testsuite log (gdb.sum and gdb.log) URL(s):
	<http://gdb-build.sergiodj.net/results/Fedora-x86_64-m32/4e/4ee89e903de920496a69c01df70db13a70a2a0be/>

*** Failed to compiled GDB.  ***
============================

+++ The full log is too big to be posted here.
+++ These are the last 100 lines of it.

In file included from ../../binutils-gdb/gdb/inferior.h:48:0,
                 from ../../binutils-gdb/gdb/infrun.c:26:
../../binutils-gdb/gdb/progspace.h:284:31: warning: *((void*)(& maybe_restore_inferior)+32).scoped_restore_current_program_space::m_saved_pspace may be used uninitialized in this function [-Wmaybe-uninitialized]
   { set_current_program_space (m_saved_pspace); }
     ~~~~~~~~~~~~~~~~~~~~~~~~~~^~~~~~~~~~~~~~~~
../../binutils-gdb/gdb/infrun.c:927:6: note: *((void*)(& maybe_restore_inferior)+32).scoped_restore_current_program_space::m_saved_pspace was declared here
      maybe_restore_inferior;
      ^~~~~~~~~~~~~~~~~~~~~~
  CXX    location.o
  CXX    m2-lang.o
  CXX    m2-typeprint.o
  CXX    m2-valprint.o
  CXX    macrocmd.o
  CXX    macroexp.o
  CXX    macroscope.o
  CXX    macrotab.o
  CXX    main.o
  CXX    maint.o
  CXX    mdebugread.o
  CXX    mem-break.o
  CXX    memattr.o
  CXX    memory-map.o
  CXX    memrange.o
  CXX    minidebug.o
  CXX    minsyms.o
  CXX    mipsread.o
  CXX    namespace.o
  CXX    objc-lang.o
  CXX    objfiles.o
  CXX    observable.o
  CXX    opencl-lang.o
  CXX    osabi.o
  CXX    osdata.o
  CXX    p-lang.o
  CXX    p-typeprint.o
  CXX    p-valprint.o
  CXX    parse.o
  CXX    printcmd.o
  CXX    probe.o
  CXX    producer.o
  CXX    progspace.o
  CXX    progspace-and-thread.o
  CXX    prologue-value.o
  CXX    psymtab.o
  CXX    record.o
  CXX    record-btrace.o
  CXX    record-full.o
  CXX    regcache.o
In file included from ../../binutils-gdb/gdb/exceptions.h:23:0,
                 from ../../binutils-gdb/gdb/utils.h:24,
                 from ../../binutils-gdb/gdb/defs.h:676,
                 from ../../binutils-gdb/gdb/record-btrace.c:22:
../../binutils-gdb/gdb/ui-out.h: In function void btrace_insn_history(ui_out*, const btrace_thread_info*, const btrace_insn_iterator*, const btrace_insn_iterator*, gdb_disassembly_flags):
../../binutils-gdb/gdb/ui-out.h:197:18: warning: asm_list.ui_out_emit_type<(ui_out_type)1>::m_uiout may be used uninitialized in this function [-Wmaybe-uninitialized]
     m_uiout->end (Type);
     ~~~~~~~~~~~~~^~~~~~
../../binutils-gdb/gdb/record-btrace.c:658:35: note: asm_list.ui_out_emit_type<(ui_out_type)1>::m_uiout was declared here
   gdb::optional<ui_out_emit_list> asm_list;
                                   ^~~~~~~~
  CXX    regcache-dump.o
  CXX    reggroups.o
  CXX    registry.o
  CXX    remote.o
  CXX    remote-fileio.o
  CXX    remote-notif.o
  CXX    reverse.o
  CXX    rust-lang.o
  CXX    sentinel-frame.o
  CXX    ser-event.o
  CXX    serial.o
  CXX    skip.o
  CXX    solib.o
  CXX    solib-target.o
  CXX    source.o
  CXX    stabsread.o
  CXX    stack.o
  CXX    std-regs.o
  CXX    symfile.o
  CXX    symfile-debug.o
  CXX    symmisc.o
  CXX    symtab.o
  CXX    target.o
../../binutils-gdb/gdb/symtab.c: In function symtab_and_line find_pc_sect_line(CORE_ADDR, obj_section*, int):
../../binutils-gdb/gdb/symtab.c:3224:5: error: this if clause does not guard... [-Werror=misleading-indentation]
     if (item != first)
     ^~
../../binutils-gdb/gdb/symtab.c:3237:7: note: ...this statement, but the latter is misleadingly indented as if it were guarded by the if
       if (prev && prev->line && (!best || prev->pc > best->pc))
       ^~
  CXX    target-dcache.o
  CXX    target-descriptions.o
  CXX    target-memory.o
cc1plus: all warnings being treated as errors
make[2]: *** [Makefile:1607: symtab.o] Error 1
make[2]: *** Waiting for unfinished jobs....
make[2]: Leaving directory '/home/gdb-buildbot-2/fedora-x86-64-4/fedora-x86-64-m32/build/gdb'
make[1]: *** [Makefile:8793: all-gdb] Error 2
make[1]: Leaving directory '/home/gdb-buildbot-2/fedora-x86-64-4/fedora-x86-64-m32/build'
make: *** [Makefile:850: all] Error 2
============================


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

* Breakage on builder Fedora-x86_64-native-extended-gdbserver-m32, revision 4ee89e903de920496a69c01df70db13a70a2a0be
  2018-03-20 14:14 Oh dear. I regret to inform you that commit 4ee89e903de920496a69c01df70db13a70a2a0be might be unfortunate gdb-buildbot
                   ` (4 preceding siblings ...)
  2018-03-20 14:17 ` Breakage on builder Fedora-x86_64-m32, " gdb-buildbot
@ 2018-03-20 14:21 ` gdb-buildbot
  2018-03-20 14:23 ` Breakage on builder Fedora-x86_64-cc-with-index, " gdb-buildbot
                   ` (3 subsequent siblings)
  9 siblings, 0 replies; 11+ messages in thread
From: gdb-buildbot @ 2018-03-20 14:21 UTC (permalink / raw)
  To: Stephen Roberts, gdb-patches

Unfortunately it seems that there is a breakage on GDB.

Commit title: 'Replace the linear search in find_pc_sect_line with a binary search.'
Revision: 4ee89e903de920496a69c01df70db13a70a2a0be

You can find more details below:

+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+

Buildslave:
	fedora-x86-64-2

Full Build URL:
	<http://gdb-build.sergiodj.net/builders/Fedora-x86_64-native-extended-gdbserver-m32/builds/9084>

Commit(s) tested:
	4ee89e903de920496a69c01df70db13a70a2a0be

Author(s) (in the same order as the commits):
	Stephen Roberts <stephen.roberts@arm.com>

Subject:
	Replace the linear search in find_pc_sect_line with a binary search.

Testsuite log (gdb.sum and gdb.log) URL(s):
	<http://gdb-build.sergiodj.net/results/Fedora-x86_64-native-extended-gdbserver-m32/4e/4ee89e903de920496a69c01df70db13a70a2a0be/>

*** Failed to compiled GDB.  ***
============================

+++ The full log is too big to be posted here.
+++ These are the last 100 lines of it.

In file included from ../../binutils-gdb/gdb/inferior.h:48:0,
                 from ../../binutils-gdb/gdb/infrun.c:26:
../../binutils-gdb/gdb/progspace.h:284:31: warning: *((void*)(& maybe_restore_inferior)+32).scoped_restore_current_program_space::m_saved_pspace may be used uninitialized in this function [-Wmaybe-uninitialized]
   { set_current_program_space (m_saved_pspace); }
     ~~~~~~~~~~~~~~~~~~~~~~~~~~^~~~~~~~~~~~~~~~
../../binutils-gdb/gdb/infrun.c:927:6: note: *((void*)(& maybe_restore_inferior)+32).scoped_restore_current_program_space::m_saved_pspace was declared here
      maybe_restore_inferior;
      ^~~~~~~~~~~~~~~~~~~~~~
  CXX    location.o
  CXX    m2-lang.o
  CXX    m2-typeprint.o
  CXX    m2-valprint.o
  CXX    macrocmd.o
  CXX    macroexp.o
  CXX    macroscope.o
  CXX    macrotab.o
  CXX    main.o
  CXX    maint.o
  CXX    mdebugread.o
  CXX    mem-break.o
  CXX    memattr.o
  CXX    memory-map.o
  CXX    memrange.o
  CXX    minidebug.o
  CXX    minsyms.o
  CXX    mipsread.o
  CXX    namespace.o
  CXX    objc-lang.o
  CXX    objfiles.o
  CXX    observable.o
  CXX    opencl-lang.o
  CXX    osabi.o
  CXX    osdata.o
  CXX    p-lang.o
  CXX    p-typeprint.o
  CXX    p-valprint.o
  CXX    parse.o
  CXX    printcmd.o
  CXX    probe.o
  CXX    producer.o
  CXX    progspace.o
  CXX    progspace-and-thread.o
  CXX    prologue-value.o
  CXX    psymtab.o
  CXX    record.o
  CXX    record-btrace.o
  CXX    record-full.o
  CXX    regcache.o
In file included from ../../binutils-gdb/gdb/exceptions.h:23:0,
                 from ../../binutils-gdb/gdb/utils.h:24,
                 from ../../binutils-gdb/gdb/defs.h:676,
                 from ../../binutils-gdb/gdb/record-btrace.c:22:
../../binutils-gdb/gdb/ui-out.h: In function void btrace_insn_history(ui_out*, const btrace_thread_info*, const btrace_insn_iterator*, const btrace_insn_iterator*, gdb_disassembly_flags):
../../binutils-gdb/gdb/ui-out.h:197:18: warning: asm_list.ui_out_emit_type<(ui_out_type)1>::m_uiout may be used uninitialized in this function [-Wmaybe-uninitialized]
     m_uiout->end (Type);
     ~~~~~~~~~~~~~^~~~~~
../../binutils-gdb/gdb/record-btrace.c:658:35: note: asm_list.ui_out_emit_type<(ui_out_type)1>::m_uiout was declared here
   gdb::optional<ui_out_emit_list> asm_list;
                                   ^~~~~~~~
  CXX    regcache-dump.o
  CXX    reggroups.o
  CXX    registry.o
  CXX    remote.o
  CXX    remote-fileio.o
  CXX    remote-notif.o
  CXX    reverse.o
  CXX    rust-lang.o
  CXX    sentinel-frame.o
  CXX    ser-event.o
  CXX    serial.o
  CXX    skip.o
  CXX    solib.o
  CXX    solib-target.o
  CXX    source.o
  CXX    stabsread.o
  CXX    stack.o
  CXX    std-regs.o
  CXX    symfile.o
  CXX    symfile-debug.o
  CXX    symmisc.o
  CXX    symtab.o
  CXX    target.o
../../binutils-gdb/gdb/symtab.c: In function symtab_and_line find_pc_sect_line(CORE_ADDR, obj_section*, int):
../../binutils-gdb/gdb/symtab.c:3224:5: error: this if clause does not guard... [-Werror=misleading-indentation]
     if (item != first)
     ^~
../../binutils-gdb/gdb/symtab.c:3237:7: note: ...this statement, but the latter is misleadingly indented as if it were guarded by the if
       if (prev && prev->line && (!best || prev->pc > best->pc))
       ^~
  CXX    target-dcache.o
  CXX    target-descriptions.o
  CXX    target-memory.o
cc1plus: all warnings being treated as errors
make[2]: *** [Makefile:1607: symtab.o] Error 1
make[2]: *** Waiting for unfinished jobs....
make[2]: Leaving directory '/home/gdb-buildbot-2/fedora-x86-64-2/fedora-x86-64-native-extended-gdbserver-m32/build/gdb'
make[1]: *** [Makefile:8793: all-gdb] Error 2
make: *** [Makefile:850: all] Error 2
make[1]: Leaving directory '/home/gdb-buildbot-2/fedora-x86-64-2/fedora-x86-64-native-extended-gdbserver-m32/build'
============================


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

* Breakage on builder Fedora-x86_64-cc-with-index, revision 4ee89e903de920496a69c01df70db13a70a2a0be
  2018-03-20 14:14 Oh dear. I regret to inform you that commit 4ee89e903de920496a69c01df70db13a70a2a0be might be unfortunate gdb-buildbot
                   ` (5 preceding siblings ...)
  2018-03-20 14:21 ` Breakage on builder Fedora-x86_64-native-extended-gdbserver-m32, " gdb-buildbot
@ 2018-03-20 14:23 ` gdb-buildbot
  2018-03-20 14:24 ` Breakage on builder Fedora-x86_64-native-extended-gdbserver-m64, " gdb-buildbot
                   ` (2 subsequent siblings)
  9 siblings, 0 replies; 11+ messages in thread
From: gdb-buildbot @ 2018-03-20 14:23 UTC (permalink / raw)
  To: Stephen Roberts, gdb-patches

Unfortunately it seems that there is a breakage on GDB.

Commit title: 'Replace the linear search in find_pc_sect_line with a binary search.'
Revision: 4ee89e903de920496a69c01df70db13a70a2a0be

You can find more details below:

+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+

Buildslave:
	fedora-x86-64-1

Full Build URL:
	<http://gdb-build.sergiodj.net/builders/Fedora-x86_64-cc-with-index/builds/9033>

Commit(s) tested:
	4ee89e903de920496a69c01df70db13a70a2a0be

Author(s) (in the same order as the commits):
	Stephen Roberts <stephen.roberts@arm.com>

Subject:
	Replace the linear search in find_pc_sect_line with a binary search.

Testsuite log (gdb.sum and gdb.log) URL(s):
	<http://gdb-build.sergiodj.net/results/Fedora-x86_64-cc-with-index/4e/4ee89e903de920496a69c01df70db13a70a2a0be/>

*** Failed to compiled GDB.  ***
============================

+++ The full log is too big to be posted here.
+++ These are the last 100 lines of it.

In file included from ../../binutils-gdb/gdb/inferior.h:48:0,
                 from ../../binutils-gdb/gdb/infrun.c:26:
../../binutils-gdb/gdb/progspace.h:284:31: warning: *((void*)(& maybe_restore_inferior)+32).scoped_restore_current_program_space::m_saved_pspace may be used uninitialized in this function [-Wmaybe-uninitialized]
   { set_current_program_space (m_saved_pspace); }
     ~~~~~~~~~~~~~~~~~~~~~~~~~~^~~~~~~~~~~~~~~~
../../binutils-gdb/gdb/infrun.c:927:6: note: *((void*)(& maybe_restore_inferior)+32).scoped_restore_current_program_space::m_saved_pspace was declared here
      maybe_restore_inferior;
      ^~~~~~~~~~~~~~~~~~~~~~
  CXX    location.o
  CXX    m2-lang.o
  CXX    m2-typeprint.o
  CXX    m2-valprint.o
  CXX    macrocmd.o
  CXX    macroexp.o
  CXX    macroscope.o
  CXX    macrotab.o
  CXX    main.o
  CXX    maint.o
  CXX    mdebugread.o
  CXX    mem-break.o
  CXX    memattr.o
  CXX    memory-map.o
  CXX    memrange.o
  CXX    minidebug.o
  CXX    minsyms.o
  CXX    mipsread.o
  CXX    namespace.o
  CXX    objc-lang.o
  CXX    objfiles.o
  CXX    observable.o
  CXX    opencl-lang.o
  CXX    osabi.o
  CXX    osdata.o
  CXX    p-lang.o
  CXX    p-typeprint.o
  CXX    p-valprint.o
  CXX    parse.o
  CXX    printcmd.o
  CXX    probe.o
  CXX    producer.o
  CXX    progspace.o
  CXX    progspace-and-thread.o
  CXX    prologue-value.o
  CXX    psymtab.o
  CXX    record.o
  CXX    record-btrace.o
  CXX    record-full.o
  CXX    regcache.o
In file included from ../../binutils-gdb/gdb/exceptions.h:23:0,
                 from ../../binutils-gdb/gdb/utils.h:24,
                 from ../../binutils-gdb/gdb/defs.h:676,
                 from ../../binutils-gdb/gdb/record-btrace.c:22:
../../binutils-gdb/gdb/ui-out.h: In function void btrace_insn_history(ui_out*, const btrace_thread_info*, const btrace_insn_iterator*, const btrace_insn_iterator*, gdb_disassembly_flags):
../../binutils-gdb/gdb/ui-out.h:197:18: warning: asm_list.ui_out_emit_type<(ui_out_type)1>::m_uiout may be used uninitialized in this function [-Wmaybe-uninitialized]
     m_uiout->end (Type);
     ~~~~~~~~~~~~~^~~~~~
../../binutils-gdb/gdb/record-btrace.c:658:35: note: asm_list.ui_out_emit_type<(ui_out_type)1>::m_uiout was declared here
   gdb::optional<ui_out_emit_list> asm_list;
                                   ^~~~~~~~
  CXX    regcache-dump.o
  CXX    reggroups.o
  CXX    registry.o
  CXX    remote.o
  CXX    remote-fileio.o
  CXX    remote-notif.o
  CXX    reverse.o
  CXX    rust-lang.o
  CXX    sentinel-frame.o
  CXX    ser-event.o
  CXX    serial.o
  CXX    skip.o
  CXX    solib.o
  CXX    solib-target.o
  CXX    source.o
  CXX    stabsread.o
  CXX    stack.o
  CXX    std-regs.o
  CXX    symfile.o
  CXX    symfile-debug.o
  CXX    symmisc.o
  CXX    symtab.o
  CXX    target.o
../../binutils-gdb/gdb/symtab.c: In function symtab_and_line find_pc_sect_line(CORE_ADDR, obj_section*, int):
../../binutils-gdb/gdb/symtab.c:3224:5: error: this if clause does not guard... [-Werror=misleading-indentation]
     if (item != first)
     ^~
../../binutils-gdb/gdb/symtab.c:3237:7: note: ...this statement, but the latter is misleadingly indented as if it were guarded by the if
       if (prev && prev->line && (!best || prev->pc > best->pc))
       ^~
  CXX    target-dcache.o
  CXX    target-descriptions.o
  CXX    target-memory.o
cc1plus: all warnings being treated as errors
make[2]: *** [Makefile:1607: symtab.o] Error 1
make[2]: *** Waiting for unfinished jobs....
make[2]: Leaving directory '/home/gdb-buildbot/fedora-x86-64-1/fedora-x86-64-cc-with-index/build/gdb'
make[1]: *** [Makefile:8793: all-gdb] Error 2
make[1]: Leaving directory '/home/gdb-buildbot/fedora-x86-64-1/fedora-x86-64-cc-with-index/build'
make: *** [Makefile:850: all] Error 2
============================


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

* Breakage on builder Fedora-x86_64-native-gdbserver-m64, revision 4ee89e903de920496a69c01df70db13a70a2a0be
  2018-03-20 14:14 Oh dear. I regret to inform you that commit 4ee89e903de920496a69c01df70db13a70a2a0be might be unfortunate gdb-buildbot
                   ` (7 preceding siblings ...)
  2018-03-20 14:24 ` Breakage on builder Fedora-x86_64-native-extended-gdbserver-m64, " gdb-buildbot
@ 2018-03-20 14:24 ` gdb-buildbot
  2018-03-20 14:29 ` Oh dear. I regret to inform you that commit 4ee89e903de920496a69c01df70db13a70a2a0be might be unfortunate Stephen Roberts
  9 siblings, 0 replies; 11+ messages in thread
From: gdb-buildbot @ 2018-03-20 14:24 UTC (permalink / raw)
  To: Stephen Roberts, gdb-patches

Unfortunately it seems that there is a breakage on GDB.

Commit title: 'Replace the linear search in find_pc_sect_line with a binary search.'
Revision: 4ee89e903de920496a69c01df70db13a70a2a0be

You can find more details below:

+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+

Buildslave:
	fedora-x86-64-4

Full Build URL:
	<http://gdb-build.sergiodj.net/builders/Fedora-x86_64-native-gdbserver-m64/builds/9078>

Commit(s) tested:
	4ee89e903de920496a69c01df70db13a70a2a0be

Author(s) (in the same order as the commits):
	Stephen Roberts <stephen.roberts@arm.com>

Subject:
	Replace the linear search in find_pc_sect_line with a binary search.

Testsuite log (gdb.sum and gdb.log) URL(s):
	<http://gdb-build.sergiodj.net/results/Fedora-x86_64-native-gdbserver-m64/4e/4ee89e903de920496a69c01df70db13a70a2a0be/>

*** Failed to compiled GDB.  ***
============================

+++ The full log is too big to be posted here.
+++ These are the last 100 lines of it.

In file included from ../../binutils-gdb/gdb/inferior.h:48:0,
                 from ../../binutils-gdb/gdb/infrun.c:26:
../../binutils-gdb/gdb/progspace.h:284:31: warning: *((void*)(& maybe_restore_inferior)+32).scoped_restore_current_program_space::m_saved_pspace may be used uninitialized in this function [-Wmaybe-uninitialized]
   { set_current_program_space (m_saved_pspace); }
     ~~~~~~~~~~~~~~~~~~~~~~~~~~^~~~~~~~~~~~~~~~
../../binutils-gdb/gdb/infrun.c:927:6: note: *((void*)(& maybe_restore_inferior)+32).scoped_restore_current_program_space::m_saved_pspace was declared here
      maybe_restore_inferior;
      ^~~~~~~~~~~~~~~~~~~~~~
  CXX    location.o
  CXX    m2-lang.o
  CXX    m2-typeprint.o
  CXX    m2-valprint.o
  CXX    macrocmd.o
  CXX    macroexp.o
  CXX    macroscope.o
  CXX    macrotab.o
  CXX    main.o
  CXX    maint.o
  CXX    mdebugread.o
  CXX    mem-break.o
  CXX    memattr.o
  CXX    memory-map.o
  CXX    memrange.o
  CXX    minidebug.o
  CXX    minsyms.o
  CXX    mipsread.o
  CXX    namespace.o
  CXX    objc-lang.o
  CXX    objfiles.o
  CXX    observable.o
  CXX    opencl-lang.o
  CXX    osabi.o
  CXX    osdata.o
  CXX    p-lang.o
  CXX    p-typeprint.o
  CXX    p-valprint.o
  CXX    parse.o
  CXX    printcmd.o
  CXX    probe.o
  CXX    producer.o
  CXX    progspace.o
  CXX    progspace-and-thread.o
  CXX    prologue-value.o
  CXX    psymtab.o
  CXX    record.o
  CXX    record-btrace.o
  CXX    record-full.o
  CXX    regcache.o
In file included from ../../binutils-gdb/gdb/exceptions.h:23:0,
                 from ../../binutils-gdb/gdb/utils.h:24,
                 from ../../binutils-gdb/gdb/defs.h:676,
                 from ../../binutils-gdb/gdb/record-btrace.c:22:
../../binutils-gdb/gdb/ui-out.h: In function void btrace_insn_history(ui_out*, const btrace_thread_info*, const btrace_insn_iterator*, const btrace_insn_iterator*, gdb_disassembly_flags):
../../binutils-gdb/gdb/ui-out.h:197:18: warning: asm_list.ui_out_emit_type<(ui_out_type)1>::m_uiout may be used uninitialized in this function [-Wmaybe-uninitialized]
     m_uiout->end (Type);
     ~~~~~~~~~~~~~^~~~~~
../../binutils-gdb/gdb/record-btrace.c:658:35: note: asm_list.ui_out_emit_type<(ui_out_type)1>::m_uiout was declared here
   gdb::optional<ui_out_emit_list> asm_list;
                                   ^~~~~~~~
  CXX    regcache-dump.o
  CXX    reggroups.o
  CXX    registry.o
  CXX    remote.o
  CXX    remote-fileio.o
  CXX    remote-notif.o
  CXX    reverse.o
  CXX    rust-lang.o
  CXX    sentinel-frame.o
  CXX    ser-event.o
  CXX    serial.o
  CXX    skip.o
  CXX    solib.o
  CXX    solib-target.o
  CXX    source.o
  CXX    stabsread.o
  CXX    stack.o
  CXX    std-regs.o
  CXX    symfile.o
  CXX    symfile-debug.o
  CXX    symmisc.o
  CXX    symtab.o
  CXX    target.o
  CXX    target-dcache.o
../../binutils-gdb/gdb/symtab.c: In function symtab_and_line find_pc_sect_line(CORE_ADDR, obj_section*, int):
../../binutils-gdb/gdb/symtab.c:3224:5: error: this if clause does not guard... [-Werror=misleading-indentation]
     if (item != first)
     ^~
../../binutils-gdb/gdb/symtab.c:3237:7: note: ...this statement, but the latter is misleadingly indented as if it were guarded by the if
       if (prev && prev->line && (!best || prev->pc > best->pc))
       ^~
  CXX    target-descriptions.o
  CXX    target-memory.o
cc1plus: all warnings being treated as errors
make[2]: *** [Makefile:1607: symtab.o] Error 1
make[2]: *** Waiting for unfinished jobs....
make[2]: Leaving directory '/home/gdb-buildbot-2/fedora-x86-64-4/fedora-x86-64-native-gdbserver/build/gdb'
make[1]: *** [Makefile:8793: all-gdb] Error 2
make[1]: Leaving directory '/home/gdb-buildbot-2/fedora-x86-64-4/fedora-x86-64-native-gdbserver/build'
make: *** [Makefile:850: all] Error 2
============================


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

* Breakage on builder Fedora-x86_64-native-extended-gdbserver-m64, revision 4ee89e903de920496a69c01df70db13a70a2a0be
  2018-03-20 14:14 Oh dear. I regret to inform you that commit 4ee89e903de920496a69c01df70db13a70a2a0be might be unfortunate gdb-buildbot
                   ` (6 preceding siblings ...)
  2018-03-20 14:23 ` Breakage on builder Fedora-x86_64-cc-with-index, " gdb-buildbot
@ 2018-03-20 14:24 ` gdb-buildbot
  2018-03-20 14:24 ` Breakage on builder Fedora-x86_64-native-gdbserver-m64, " gdb-buildbot
  2018-03-20 14:29 ` Oh dear. I regret to inform you that commit 4ee89e903de920496a69c01df70db13a70a2a0be might be unfortunate Stephen Roberts
  9 siblings, 0 replies; 11+ messages in thread
From: gdb-buildbot @ 2018-03-20 14:24 UTC (permalink / raw)
  To: Stephen Roberts, gdb-patches

Unfortunately it seems that there is a breakage on GDB.

Commit title: 'Replace the linear search in find_pc_sect_line with a binary search.'
Revision: 4ee89e903de920496a69c01df70db13a70a2a0be

You can find more details below:

+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+

Buildslave:
	fedora-x86-64-3

Full Build URL:
	<http://gdb-build.sergiodj.net/builders/Fedora-x86_64-native-extended-gdbserver-m64/builds/9124>

Commit(s) tested:
	4ee89e903de920496a69c01df70db13a70a2a0be

Author(s) (in the same order as the commits):
	Stephen Roberts <stephen.roberts@arm.com>

Subject:
	Replace the linear search in find_pc_sect_line with a binary search.

Testsuite log (gdb.sum and gdb.log) URL(s):
	<http://gdb-build.sergiodj.net/results/Fedora-x86_64-native-extended-gdbserver-m64/4e/4ee89e903de920496a69c01df70db13a70a2a0be/>

*** Failed to compiled GDB.  ***
============================

+++ The full log is too big to be posted here.
+++ These are the last 100 lines of it.

In file included from ../../binutils-gdb/gdb/inferior.h:48:0,
                 from ../../binutils-gdb/gdb/infrun.c:26:
../../binutils-gdb/gdb/progspace.h:284:31: warning: *((void*)(& maybe_restore_inferior)+32).scoped_restore_current_program_space::m_saved_pspace may be used uninitialized in this function [-Wmaybe-uninitialized]
   { set_current_program_space (m_saved_pspace); }
     ~~~~~~~~~~~~~~~~~~~~~~~~~~^~~~~~~~~~~~~~~~
../../binutils-gdb/gdb/infrun.c:927:6: note: *((void*)(& maybe_restore_inferior)+32).scoped_restore_current_program_space::m_saved_pspace was declared here
      maybe_restore_inferior;
      ^~~~~~~~~~~~~~~~~~~~~~
  CXX    location.o
  CXX    m2-lang.o
  CXX    m2-typeprint.o
  CXX    m2-valprint.o
  CXX    macrocmd.o
  CXX    macroexp.o
  CXX    macroscope.o
  CXX    macrotab.o
  CXX    main.o
  CXX    maint.o
  CXX    mdebugread.o
  CXX    mem-break.o
  CXX    memattr.o
  CXX    memory-map.o
  CXX    memrange.o
  CXX    minidebug.o
  CXX    minsyms.o
  CXX    mipsread.o
  CXX    namespace.o
  CXX    objc-lang.o
  CXX    objfiles.o
  CXX    observable.o
  CXX    opencl-lang.o
  CXX    osabi.o
  CXX    osdata.o
  CXX    p-lang.o
  CXX    p-typeprint.o
  CXX    p-valprint.o
  CXX    parse.o
  CXX    printcmd.o
  CXX    probe.o
  CXX    producer.o
  CXX    progspace.o
  CXX    progspace-and-thread.o
  CXX    prologue-value.o
  CXX    psymtab.o
  CXX    record.o
  CXX    record-btrace.o
  CXX    record-full.o
  CXX    regcache.o
In file included from ../../binutils-gdb/gdb/exceptions.h:23:0,
                 from ../../binutils-gdb/gdb/utils.h:24,
                 from ../../binutils-gdb/gdb/defs.h:676,
                 from ../../binutils-gdb/gdb/record-btrace.c:22:
../../binutils-gdb/gdb/ui-out.h: In function void btrace_insn_history(ui_out*, const btrace_thread_info*, const btrace_insn_iterator*, const btrace_insn_iterator*, gdb_disassembly_flags):
../../binutils-gdb/gdb/ui-out.h:197:18: warning: asm_list.ui_out_emit_type<(ui_out_type)1>::m_uiout may be used uninitialized in this function [-Wmaybe-uninitialized]
     m_uiout->end (Type);
     ~~~~~~~~~~~~~^~~~~~
../../binutils-gdb/gdb/record-btrace.c:658:35: note: asm_list.ui_out_emit_type<(ui_out_type)1>::m_uiout was declared here
   gdb::optional<ui_out_emit_list> asm_list;
                                   ^~~~~~~~
  CXX    regcache-dump.o
  CXX    reggroups.o
  CXX    registry.o
  CXX    remote.o
  CXX    remote-fileio.o
  CXX    remote-notif.o
  CXX    reverse.o
  CXX    rust-lang.o
  CXX    sentinel-frame.o
  CXX    ser-event.o
  CXX    serial.o
  CXX    skip.o
  CXX    solib.o
  CXX    solib-target.o
  CXX    source.o
  CXX    stabsread.o
  CXX    stack.o
  CXX    std-regs.o
  CXX    symfile.o
  CXX    symfile-debug.o
  CXX    symmisc.o
  CXX    symtab.o
  CXX    target.o
  CXX    target-dcache.o
../../binutils-gdb/gdb/symtab.c: In function symtab_and_line find_pc_sect_line(CORE_ADDR, obj_section*, int):
../../binutils-gdb/gdb/symtab.c:3224:5: error: this if clause does not guard... [-Werror=misleading-indentation]
     if (item != first)
     ^~
../../binutils-gdb/gdb/symtab.c:3237:7: note: ...this statement, but the latter is misleadingly indented as if it were guarded by the if
       if (prev && prev->line && (!best || prev->pc > best->pc))
       ^~
  CXX    target-descriptions.o
  CXX    target-memory.o
cc1plus: all warnings being treated as errors
make[2]: *** [Makefile:1607: symtab.o] Error 1
make[2]: *** Waiting for unfinished jobs....
make[2]: Leaving directory '/home/gdb-buildbot/fedora-x86-64-3/fedora-x86-64-native-extended-gdbserver/build/gdb'
make[1]: Leaving directory '/home/gdb-buildbot/fedora-x86-64-3/fedora-x86-64-native-extended-gdbserver/build'
make[1]: *** [Makefile:8793: all-gdb] Error 2
make: *** [Makefile:850: all] Error 2
============================


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

* Re: Oh dear. I regret to inform you that commit 4ee89e903de920496a69c01df70db13a70a2a0be might be unfortunate
  2018-03-20 14:14 Oh dear. I regret to inform you that commit 4ee89e903de920496a69c01df70db13a70a2a0be might be unfortunate gdb-buildbot
                   ` (8 preceding siblings ...)
  2018-03-20 14:24 ` Breakage on builder Fedora-x86_64-native-gdbserver-m64, " gdb-buildbot
@ 2018-03-20 14:29 ` Stephen Roberts
  9 siblings, 0 replies; 11+ messages in thread
From: Stephen Roberts @ 2018-03-20 14:29 UTC (permalink / raw)
  To: gdb-buildbot, gdb-patches; +Cc: nd

Apologies, I'm looking into this as a matter of urgency. It appears that my patch has caused the indentation of the following block to become incorrect.

Sorry, 
Stephen R.

---------
From: gdb-buildbot@sergiodj.net <gdb-buildbot@sergiodj.net>
Sent: 20 March 2018 14:14
To: Stephen Roberts; gdb-patches@sourceware.org
Subject: Oh dear. I regret to inform you that commit 4ee89e903de920496a69c01df70db13a70a2a0be might be unfortunate
  

My lords, ladies, gentlemen, members of the public.

It is a matter of great regret and sadness to inform you that commit:

        Replace the linear search in find_pc_sect_line with a binary search.
        4ee89e903de920496a69c01df70db13a70a2a0be

might have made GDB unwell.  Since I am just your Butler BuildBot,
I kindly ask that a human superior officer double-check this.

Please note that if you are reading this message on gdb-patches, there might
be other builders broken.

You can find more details about the unfortunate breakage in the next messages.

Cheers,

Your GDB BuildBot.
    
From gdb-patches-return-146050-listarch-gdb-patches=sources.redhat.com@sourceware.org Tue Mar 20 15:11:45 2018
Return-Path: <gdb-patches-return-146050-listarch-gdb-patches=sources.redhat.com@sourceware.org>
Delivered-To: listarch-gdb-patches@sources.redhat.com
Received: (qmail 117680 invoked by alias); 20 Mar 2018 15:11:44 -0000
Mailing-List: contact gdb-patches-help@sourceware.org; run by ezmlm
Precedence: bulk
List-Id: <gdb-patches.sourceware.org>
List-Subscribe: <mailto:gdb-patches-subscribe@sourceware.org>
List-Archive: <http://sourceware.org/ml/gdb-patches/>
List-Post: <mailto:gdb-patches@sourceware.org>
List-Help: <mailto:gdb-patches-help@sourceware.org>, <http://sourceware.org/ml/#faqs>
Sender: gdb-patches-owner@sourceware.org
Delivered-To: mailing list gdb-patches@sourceware.org
Received: (qmail 117670 invoked by uid 89); 20 Mar 2018 15:11:44 -0000
Authentication-Results: sourceware.org; auth=none
X-Virus-Found: No
X-Spam-SWARE-Status: No, score=-2.8 required=5.0 tests=AWL,BAYES_00,T_RP_MATCHES_RCVD autolearn=ham version=3.3.2 spammy=HContent-Transfer-Encoding:8bit
X-HELO: mx1.redhat.com
Received: from mx3-rdu2.redhat.com (HELO mx1.redhat.com) (66.187.233.73) by sourceware.org (qpsmtpd/0.93/v0.84-503-g423c35a) with ESMTP; Tue, 20 Mar 2018 15:11:43 +0000
Received: from smtp.corp.redhat.com (int-mx03.intmail.prod.int.rdu2.redhat.com [10.11.54.3])	(using TLSv1.2 with cipher AECDH-AES256-SHA (256/256 bits))	(No client certificate requested)	by mx1.redhat.com (Postfix) with ESMTPS id BE7B180AD238;	Tue, 20 Mar 2018 15:11:41 +0000 (UTC)
Received: from [127.0.0.1] (ovpn04.gateway.prod.ext.ams2.redhat.com [10.39.146.4])	by smtp.corp.redhat.com (Postfix) with ESMTP id 16E2410B0F5B;	Tue, 20 Mar 2018 15:11:40 +0000 (UTC)
Subject: Re: [RFA v2] (x86) Fix watchpoint using hardware breakpoint for some distro
To: Xavier Roirand <roirand@adacore.com>, gdb-patches@sourceware.org
References: <1521209212-11264-1-git-send-email-roirand@adacore.com> <b8442988-85e3-dc87-0e4c-732395b8db93@redhat.com> <9c7c8586-2940-bea9-d3fb-a13b0d38a32e@adacore.com>
Cc: brobecker@adacore.com
From: Pedro Alves <palves@redhat.com>
Message-ID: <c0d80d21-9f0e-c6b0-caaf-7b6246e83807@redhat.com>
Date: Tue, 20 Mar 2018 15:11:00 -0000
User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:52.0) Gecko/20100101 Thunderbird/52.6.0
MIME-Version: 1.0
In-Reply-To: <9c7c8586-2940-bea9-d3fb-a13b0d38a32e@adacore.com>
Content-Type: text/plain; charset=utf-8
Content-Transfer-Encoding: 8bit
X-SW-Source: 2018-03/txt/msg00379.txt.bz2
Content-length: 1151

On 03/20/2018 02:28 PM, Xavier Roirand wrote:
> Hello Pedro,
>
> I've replied to your comments and attached a v2 patch.
>
> Le 3/19/18 à 3:28 PM, Pedro Alves a écrit :
>> A few things are missing here:
>>
>> #1 - kernel versions where this was observed.
>>
>
> CentOS: 2.6.18-419.el5
> Suse: 2.6.27.19-5-pae
>
>> #2 - If it's not equal to TRAP_HWBKPT, then what's it equal to?
>>       I assume zero?
>
> No, it's equal to 1.

Hmm, that's TRAP_BRKPT nowadays.  What was '1' supposed to mean in
kernels of such vintage?  What was it's symbolic name back then?

In the table in linux-ptrace.h, we see that modern kernels report
TRAP_BRKPT/1 for the "single-stepping a syscall" case.  What do
those older kernels report in that case then?

>> Does the step-into-watchpoint case result in TRAP_TRACE, or does
>> that result in 0 too?  That affects the "continue" in the comment above.
>
> This results to a value of 1 too.

What do those kernels report for hardware _breakpoints_?  Is it 1 too?
Wondering whether we should make GDB_ARCH_IS_TRAP_HWBKPT return
true for 1 too...

Please provide a more complete picture.

Thanks,
Pedro Alves


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

end of thread, other threads:[~2018-03-20 14:29 UTC | newest]

Thread overview: 11+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2018-03-20 14:14 Oh dear. I regret to inform you that commit 4ee89e903de920496a69c01df70db13a70a2a0be might be unfortunate gdb-buildbot
2018-03-20 14:14 ` Breakage on builder Fedora-i686, revision 4ee89e903de920496a69c01df70db13a70a2a0be gdb-buildbot
2018-03-20 14:16 ` Breakage on builder Fedora-x86_64-native-gdbserver-m32, " gdb-buildbot
2018-03-20 14:16 ` Breakage on builder Fedora-x86_64-m64, " gdb-buildbot
2018-03-20 14:17 ` Breakage on builder Fedora-s390x-m64, " gdb-buildbot
2018-03-20 14:17 ` Breakage on builder Fedora-x86_64-m32, " gdb-buildbot
2018-03-20 14:21 ` Breakage on builder Fedora-x86_64-native-extended-gdbserver-m32, " gdb-buildbot
2018-03-20 14:23 ` Breakage on builder Fedora-x86_64-cc-with-index, " gdb-buildbot
2018-03-20 14:24 ` Breakage on builder Fedora-x86_64-native-extended-gdbserver-m64, " gdb-buildbot
2018-03-20 14:24 ` Breakage on builder Fedora-x86_64-native-gdbserver-m64, " gdb-buildbot
2018-03-20 14:29 ` Oh dear. I regret to inform you that commit 4ee89e903de920496a69c01df70db13a70a2a0be might be unfortunate Stephen Roberts

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