and though bugs are the bane of my existence, rest assured the wretched thing will get the best of care here

  1. 25 Jul, 2016 1 commit
  2. 21 Jul, 2016 1 commit
  3. 11 Jul, 2016 6 commits
  4. 10 Jul, 2016 1 commit
  5. 09 Jul, 2016 2 commits
  6. 08 Jul, 2016 4 commits
    • jamesbowman's avatar
      FT32: Correct 32-bit reloc for BFD_RELOC_32 · 458653a9
      jamesbowman authored
      The reloc for BFD_RELOC_32 was using the the 20-bit. This hack causes
      problems in gdb.  Fixed it to be the proper 32-bit reloc, R_FT32_32.
      
      bfd/ChangeLog:
      
      	* elf32-ft32.c (ft32_reloc_map): Use R_FT32_32 for BFD_RELOC_32.
      458653a9
    • jamesbowman's avatar
      FT32: adjust disassembly opcode match fields · 2f831b9a
      jamesbowman authored
      Tighten up the opcode match fields for conditional jump and call
      instructions so more general opcodes don't match them in disassembly.
      
      opcodes/ChangeLog:
      
      	* opcodes/ft32-opc.c (ft32_opc_info): Correct mask for "callc"
      	and "jmpc".
      2f831b9a
    • Maciej W. Rozycki's avatar
      MIPS/GAS: Remove extraneous `install_insn' call from `append_insn' · b8bca85b
      Maciej W. Rozycki authored
      Complement:
      
      commit 1e915849
      Author: Richard Sandiford <rdsandiford@googlemail.com>
      Date:   Wed Mar 9 09:17:02 2005 +0000
      
      <https://sourceware.org/ml/binutils/2005-03/msg00217.html>, ("Rework
      MIPS nop-insertion code, add -mfix-vr4130 [5/11]"), and remove a call to
      `install_insn' from `append_insn', which as from that change has become
      redundant.  This is because such a call, to place an instruction's bit
      pattern in output, is already made from `move_insn', called from
      `add_relaxed_insn' or `add_fixed_insn' as appropriate, either of which
      now always is and has to be made from `append_insn' before the repeated
      call to `install_insn' is made.  Previously the place where this second
      invocation is made was the only one where the output stream was updated,
      although the update was made inline rather than with a function call.
      
      Remove the repeated call then, to reclaim some performance.
      
      	gas/
      	* config/tc-mips.c (append_insn): Remove extraneous
      	`install_insn' call.
      b8bca85b
    • GDB Administrator's avatar
      Automatic date update in version.in · fc1b2779
      GDB Administrator authored
      fc1b2779
  7. 07 Jul, 2016 4 commits
    • Walfred Tedeschi's avatar
      [obv] Fix broken build on Fedora 23. · 41c977aa
      Walfred Tedeschi authored
      Compiler complains about possible utilization of "symbol" which is member
      of lang_def.
      Initialization was added.
      
      2016-07-07  Walfred Tedeschi  <walfred.tedeschi@intel.com>
      
      gdb/ChangeLog:
      
      	* cp-namespace.c (cp_lookup_bare_symbol): Initialize
      	lang_this.symbol.
      41c977aa
    • Walfred Tedeschi's avatar
      Fix of default lookup for "this" symbol. · 4f19a0e6
      Walfred Tedeschi authored
      Using the default lookup for the symbol "this" might lead to segmentation
      fault in GDB.
      Some languages, e.g. Fortran, use as default lookup routine the C++
      routines.
      For those languages "this" can be the instance of a class or even the
      definition of a class.
      When an instance of a class having the name "this" is evaluated
      in GDB a segmentation fault was observed.
      
      As example of the issue take into consideration the Fortran code:
        type foo
          real :: a
          type(bar) :: x
          character*7 :: b
        end type foo
        type(foo) :: this
      
      Issue appears when evaluating the variable "this" in GDB.
      
      Within the language definition structure there is a field that represents
      the name of the special symbol used for the C++ "this" for the language
      being described.
      The fix presented here takes into account the aforementioned field. In the
      case the aforementioned field is NULL "this" is not represented in the
      language described and the lookup should return a null_block_symbol.
      
      Tests: Performed tests with gfortran and ifort.
      
      Reviewed:
      https://sourceware.org/ml/gdb-patches/2016-04/msg00068.html
      
      After the commited patch:
      https://sourceware.org/ml/gdb-patches/2016-06/msg00364.html
      Patch can be applied.
      
      2016-06-16  Walfred Tedeschi  <walfred.tedeschi@intel.com>
      
      gdb/ChangeLog:
      
      	* cp-namespace.c (cp_lookup_bare_symbol): Use language passed as
      	parameter to look for the symbol "this".
      
      gdb/testsuite/ChangeLog:
      
      	* gdb.fortran/derived-types.exp (result_line, result_line_2):
      	New variables.
      	(print this%a, print this%b, print this): New tests.
      	* gdb.fortran/derived-types.f90 (this): New object and
      	initialization.
      4f19a0e6
    • jamesbowman's avatar
      FT32 linker script cleanup · 3a5ce950
      jamesbowman authored
      Fix a typo (__PMSIZE was written as __PMSIZE_) and add section alignment
      for DATA and BSS.
      
      ld/ChangeLog:
      
        * scripttempl/ft32.sc (__PMSIZE): Correct __PMSIZE_.
          (DATA): add ALIGN.
          (BSS): add ALIGN
      3a5ce950
    • GDB Administrator's avatar
      Automatic date update in version.in · 819490e2
      GDB Administrator authored
      819490e2
  8. 06 Jul, 2016 10 commits
    • Simon Marchi's avatar
      gdb.ada/arraydim.exp: Fix directory layout · 986cf455
      Simon Marchi authored
      I forgot to fix this one in the previous commit.
      
      gdb/testsuite/ChangeLog:
      
      	* gdb.ada/arraydim.exp: Remove extra directory level in build
      	directory.
      986cf455
    • Simon Marchi's avatar
      Remove extra output directory level for Ada tests · f0464b23
      Simon Marchi authored
      The output of Ada tests create a layout where the test name
      ("formatted_ref" in this example) appears twice:
      
      outputs
      └── gdb.ada
          └── formatted_ref
              └── formatted_ref
                  ├── b~formatted_ref.adb
                  ├── b~formatted_ref.ads
                  ├── b~formatted_ref.ali
                  ├── b~formatted_ref.o
                  ├── defs.ali
                  ├── defs.o
                  ├── formatted_ref
                  ├── formatted_ref.ali
                  └── formatted_ref.o
      
      This causes a problem when testing with the native-gdbserver board, when
      the binary has the same name as the test.  When gdb_remote_download is
      called to upload the compiled binary, the implementation for
      native-gdbserver copies it in the standard output directory (in
      outputs/gdb.ada/formatted_ref).  However, there is already a directory
      named formatted_ref in there, so the copy fails and gdbserver isn't able
      to load the binary.
      
      This patch bypasses the problem by removing the extra directory level.
      The compiled binary will already be in its final location in the
      standard output directory, so the copy will effectively be a no-op.
      
      gdb/testsuite/ChangeLog:
      
      	* lib/ada.exp: Remove extra directory level in build directory.
      	* gdb.ada/cond_lang.exp: Likewise.
      	* gdb.ada/exec_changed.exp: Likewise.
      	* gdb.ada/lang_switch.exp: Likewise.
      f0464b23
    • John Baldwin's avatar
      Remove extraneous parentheses. · fb36c6bf
      John Baldwin authored
      gdb/ChangeLog:
      
      	* h8300-tdep.c (h8300_print_register): Remove extraneous parentheses.
      fb36c6bf
    • John Baldwin's avatar
      Use unsigned integer constant with left shifts. · db297a65
      John Baldwin authored
      This avoids undefined behavior.
      
      gdb/ChangeLog:
      
      	* ada-lang.c (ada_unpack_from_contents): Use unsigned constants with
      	left shifts.
      db297a65
    • John Baldwin's avatar
      Set uses_fp for frames with a valid FP register explicitly. · 9ca10714
      John Baldwin authored
      Since CORE_ADDR is unsigned, the saved FP register is always greater than
      or equal to zero.  Replace the comparison by explicitly setting uses_fp to
      1 for frames with a valid FP register.
      
      gdb/ChangeLog:
      
      	* sh64-tdep.c (sh64_analyze_prologue): Set "uses_fp" when setting
      	the MEDIA_FP_REGNUM register.
      9ca10714
    • John Baldwin's avatar
      Remove check for negative size. · d66ff635
      John Baldwin authored
      Since CORE_ADDR is unsigned, this value can never be negative.
      
      gdb/ChangeLog:
      
      	* score-tdep.c (score7_malloc_and_get_memblock): Remove check for
      	negative size.
      d66ff635
    • John Baldwin's avatar
      Use 'ptid_t' instead of 'ptid' for fbsd_next_vfork_done's return type. · ee950322
      John Baldwin authored
      'ptid' compiles in C++, but not C.
      
      gdb/ChangeLog:
      	* fbsd-nat.c (fbsd_is_vfork_done_pending): Fix return type.
      ee950322
    • Yao Qi's avatar
      [ARM] Fix endless recursion on calculating CPRC candidate · 1040b979
      Yao Qi authored
      When GDB determines whether type T can be part of candidate for
      passing and returning in VFP registers, it calls
      arm_vfp_cprc_sub_candidate recursively.  However, if type T has
      self-reference field, like,
      
      class C
      {
        static C s;
      };
      
      arm_vfp_cprc_sub_candidate won't return.  This fix is to skip calling
      arm_vfp_cprc_sub_candidate if the field is static.
      
      gdb:
      
      2016-07-06  Yao Qi  <yao.qi@linaro.org>
      
      	* arm-tdep.c (arm_vfp_cprc_sub_candidate): Don't call
      	arm_vfp_cprc_sub_candidate for static field.
      1040b979
    • Manish Goregaokar's avatar
      Allow subscripting raw pointers · 42d94011
      Manish Goregaokar authored
      This will be useful for dealing with vectors; regardless of our final solution
      for the Index trait.
      
      2016-07-06  Manish Goregaokar  <manish@mozilla.com>
      
      gdb/ChangeLog:
          * rust-lang.c (rust_subscript): Allow subscripting pointers
      
      gdb/testsuite/ChangeLog:
          * simple.rs: Add test for raw pointer subscripting
          * simple.exp: Add test expectations
      42d94011
    • GDB Administrator's avatar
      Automatic date update in version.in · 6949d8a7
      GDB Administrator authored
      6949d8a7
  9. 05 Jul, 2016 8 commits
    • H.J. Lu's avatar
      Add -flto to PR ld/20321 test · 29d7478b
      H.J. Lu authored
      Before GCC 4.9, -flto is required for final LTO link.  Add -flto to PR
      ld/20321 test to support older versions of GCC.
      
      	* testsuite/ld-plugin/lto.exp: Add -flto to PR ld/20321 test.
      29d7478b
    • Yao Qi's avatar
      Fix fail in gdb.mi/mi-reverse.exp · 647c264c
      Yao Qi authored
      Commit 38b022b4 adds "method" and
      "format" fields in =record-started, but doesn't update test case
      gdb.mi/mi-reverse.exp, so it causes the fail like this,
      
      PASS: gdb.mi/mi-reverse.exp: mi runto main
      Expecting: ^(-interpreter-exec console record[^M
      ]+)?(=record-started,thread-group="i1"^M
      \^done[^M
      ]+[(]gdb[)] ^M
      [ ]*)
      -interpreter-exec console record^M
      =record-started,thread-group="i1",method="full"^M
      ^done^M
      (gdb) ^M
      FAIL: gdb.mi/mi-reverse.exp: Turn on process record
      
      and regression was found by buildbot too
      https://sourceware.org/ml/gdb-testers/2016-q2/msg04492.html
      
      gdb/testsuite:
      
      2016-07-05  Yao Qi  <yao.qi@linaro.org>
      
      	* gdb.mi/mi-reverse.exp: Match =record-started output.
      647c264c
    • Andre Vieria's avatar
      [ARM] Purecode compatible long branch veneer for M-profile targets with MOVW. · d5a67c02
      Andre Vieria authored
      2016-07-05  Andre Vieria  <andre.simoesdiasvieira@arm.com>
      
      	* elf32-arm.c (THUMB32_MOVT): New veneer macro.
      	(THUMB32_MOVW): Likewise.
      	(elf32_arm_stub_long_branch_thumb2_only_pure): New.
      	(DEF_STUBS): Define long_branch_thumb2_only_pure.
      	(arm_stub_is_thumb): Add new veneer stub.
      	(arm_type_of_stub): Use new veneer.
      	(arm_stub_required_alignment): Add new veneer.
      
      2016-07-05  Andre Vieria  <andre.simoesdiasvieira@arm.com>
      
      	* testsuite/ld-arm/farcall-thumb2-purecode.d: New test result.
      	* testsuite/ld-arm/farcall-thumb2-purecode.s: New test.
      	* testsuite/ld-arm/arm-elf.exp: Run it.
      d5a67c02
    • Andre Vieria's avatar
      [ARM] Change noread to purecode. · f0728ee3
      Andre Vieria authored
      bfd/ChangeLog
      2016-07-05  Andre Vieria  <andre.simoesdiasvieira@arm.com>
      
              * bfd-in2.h (SEC_ELF_NOREAD): Rename to ...
                (SEC_ELF_PURECODE): ... this.
              * elf32-arm.c (elf32_arm_post_process_headers): Rename SEC_ELF_NOREAD
                to SEC_ELF_NOREAD.
                (elf32_arm_fake_sections): Likewise.
                (elf_32_arm_section_flags): Likewise.
                (elf_32_arm_lookup_section_flags): Likewise.
              * section.c (SEC_ELF_NOREAD): Rename to ...
                (SEC_ELF_PURECODE): ... this.
      
      binutils/ChangeLog
      2016-07-05  Andre Vieria  <andre.simoesdiasvieira@arm.com>
      
              * objdump.c (dump_section_header): Rename SEC_ELF_NOREAD
                to SEC_ELF_NOREAD.
              * readelf.c (get_elf_section_flags): Rename ARM_NOREAD to
                ARM_PURECODE and SHF_ARM_NOREAD to SHF_ARM_PURECODE.
                (process_section_headers): Rename noread to purecode.
      
              * section.c (SEC_ELF_NOREAD): Rename to ...
                (SEC_ELF_PURECODE): ... this.
      
      include/ChangeLog
      2016-07-05  Andre Vieria  <andre.simoesdiasvieira@arm.com>
      
              * elf/arm.h (SHF_ARM_NOREAD): Rename to ...
                (SHF_ARM_PURECODE): ... this.
      
      ld/ChangeLog
      2016-07-05  Andre Vieria  <andre.simoesdiasvieira@arm.com>
      
              * testsuite/ld-arm/arm_noread.ld: Renamed to ...
                testsuite/ld-arm/arm_purecode.ld: ... this, and replaced
                all noread's by purecode.
      f0728ee3
    • Jan Beulich's avatar
      ld: track linker-definedness of symbols · 1753ed68
      Jan Beulich authored
      Keep "lineno" as zero while not processing any script, and use it being
      zero to set the "linker_def" field to true.
      1753ed68
    • Jan Beulich's avatar
      x86: fix register check in check_qword_reg() · 33d0ab95
      Jan Beulich authored
      A missing 'r' (or wrong 'e') register prefix needs to be complained
      about if the template allows for a 64-bit register, not a 32-bit one.
      I assume this was a copy-and-paste type of mistake
      (from check_long_reg()).
      33d0ab95
    • Jan Kratochvil's avatar
      babeltrace compilation regression · 13cdc2af
      Jan Kratochvil authored
      Since:
      	commit 2d681be4
      	Author: Andreas Arnez <arnez@linux.vnet.ibm.com>
      	Date:   Wed Apr 27 15:52:16 2016 +0200
      	    Avoid non-C++-enabled babeltrace versions
      tested with:
      	libbabeltrace-devel-1.2.4-4.fc24.x86_64
      	libbabeltrace-devel-1.4.0-2.fc25.x86_64
      it can no longer build due to:
      	configure:16435: gcc -o conftest -m64 -g3 -pipe -Wall -fexceptions -fstack-protector-strong --param=ssp-buffer-size=4 -fno-diagno
      stics-show-caret  -Werror  -static-libstdc++ -static-libgcc  conftest.c -ldl -ldl -lncurses -lm -ldl  -lbabeltrace -lbabeltrace-ctf >&5
      	conftest.c: In function 'main':
      	conftest.c:208:7: error: 'pos' is a pointer; did you mean to use '->'?
      
      gdb/ChangeLog
      2016-07-05  Jan Kratochvil  <jan.kratochvil@redhat.com>
      
      	* configure: Regenerate.
      	* configure.ac (HAVE_LIBBABELTRACE): Fix pos variable dereference.
      13cdc2af
    • GDB Administrator's avatar
      Automatic date update in version.in · 0a2f0f5d
      GDB Administrator authored
      0a2f0f5d
  10. 04 Jul, 2016 3 commits