summaryrefslogtreecommitdiffstats
path: root/binutils-2.25
Commit message (Collapse)AuthorAgeFilesLines
* Merge branch 'master' of https://android.googlesource.com/toolchain/binutilsHEADreplicant-6.0-0004-rc3replicant-6.0-0004-rc2replicant-6.0-0004-rc1replicant-6.0-0003replicant-6.0-0002replicant-6.0-0001masterWolfgang Wiedmeyer2017-05-021-8/+8
|\
| * Fix bug where -u option with empty archive results in internal error.Cary Coutant2017-04-131-8/+8
| | | | | | | | | | | | | | | | | | | | gold/ PR gold/20693 * gold.cc (queue_middle_tasks): Force valid target earlier. This change is backport of commit c13b08dbbcf752480a7621aa1f89bc5440c74c25 - https://sourceware.org/git/?p=binutils-gdb.git;a=commit;h=c13b08dbbcf752480a7621aa1f89bc5440c74c25 to binutils 2.25.
* | Merge branch 'master' of https://android.googlesource.com/toolchain/binutilsWolfgang Wiedmeyer2017-02-115-22/+47
|\ \ | |/
| * Set CodeView pdb path/name to base namerefs/keep-around/9bf6cbf6dd88777719c5757922aff0190ec46539Joshua Lang2017-01-315-22/+47
| | | | | | | | | | | | | | | | | | The CodeView pdb path/name field is empty leading windows minidumps to have empty module paths for the binary. This patch sets the path to the base name of the binary. Bug: 33668347 Change-Id: I4138998239b07b54cc36dc2d2e3484c4b63f5435
* | Merge branch 'master' of https://android.googlesource.com/toolchain/binutilsWolfgang Wiedmeyer2016-12-086-130/+236
|\ \ | |/
| * gold: enable new dtags by default.refs/keep-around/082ed0f10cf59b53381cefda2f90247e2a81015bRahul Chaudhry2016-11-031-1/+1
| | | | | | | | | | | | | | This is also the default upstream: https://sourceware.org/git/gitweb.cgi?p=binutils-gdb.git;a=commit;h=50701cc170a7c10a72e5ec2c7c6ed31057a31d43 Bug: 31270843
| * gold: make icf respect section alignment requirements.refs/keep-around/b8ff982870858c54e09105458b6955b391cc0a11Rahul Chaudhry2016-10-215-129/+235
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Bug: https://bugs.chromium.org/p/chromium/issues/detail?id=628795 The upstream bug is https://sourceware.org/bugzilla/show_bug.cgi?id=17704 This patch is backported from upstream - commit ac423761af22f7858a1413cda5df3e1d5e88d4e4 Author: Gergely Nagy <ngg@tresorit.com> Date: Fri Oct 21 11:08:20 2016 -0700 Fix PR 17704. This fix keeps the section with the highest alignment when folding sections with ICF. PR gold/17704 * icf.cc (match_sections): Add new parameter section_addraligns. Check section alignment and keep the section with the strictest alignment. (find_identical_sections): New local variable section_addraligns. Store each section's alignment. * testsuite/pr17704a_test.s: New file. * testsuite/Makefile.am (pr17704a_test): New test. * testsuite/Makefile.in: Regenerate.
* | various build fixes for Debian StretchWolfgang Wiedmeyer2016-12-076-50/+3
| | | | | | | | Signed-off-by: Wolfgang Wiedmeyer <wolfgit@wiedmeyer.de>
* | Merge branch 'master' of https://android.googlesource.com/toolchain/binutilsWolfgang Wiedmeyer2016-09-0838-117/+513
|\ \ | |/
| * Remove warning about references from shared objects to hidden symbols.Rahul Chaudhry2016-06-285-20/+35
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Bug: 21502954 This patch is backported from upstream - commit c20ceeb226168ffd84078ef74d890c2b7f69a435 Author: Yiran Wang <yiran@google.com> Date: Mon Jul 20 08:47:57 2015 -0700 Remove warning about references from shared objects to hidden symbols. gold/ PR gold/15574 * resolve.cc (Symbol_table): Remove warning about references from shared objects to hidden symbols. * testsuite/Makefile.am (hidden_test): Add hidden_test.syms. * testsuite/Makefile.in: Regenerate. * testsuite/hidden_test.sh: Check dynamic symbol table; update expected error messages. Change-Id: Ie6fb67071c7f478859069b7b4bf629cd44f91844
| * Revert "Disable the warning of a hidden symbol matches a reference to be ↵Rahul Chaudhry2016-06-282-2/+18
| | | | | | | | | | | | | | | | | | | | | | | | resolved." Bug: 21502954 This change reverts https://android-review.googlesource.com/#/c/155101/, to be replaced by the upstream patch with tests. - https://sourceware.org/git/gitweb.cgi?p=binutils-gdb.git;a=commitdiff;h=c20ceeb226168ffd84078ef74d890c2b7f69a435 Change-Id: Id929c8422a016638f860ac508eeafc7080584600
| * GOLD/DWP: exit without segfault if the binary is not built with debug fission.Yunlian Jiang2016-05-171-0/+4
| | | | | | | | | | | | | | | | | | | | | | dwp segfaults when trying to get .dwp file from a binary built without debug fission. This patch fixes that. upstream thread is https://sourceware.org/ml/binutils/2016-05/msg00204.html BUG=None TEST=dwp still works with files built with debug fission and dwp does not segfault on files that are not built with debug fission.
| * Fix DT_MIPS_RLD_MAP_REL tag for n64 target and 32-bit host.Joseph Myers2016-04-062-1/+7
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Cherry-picked from upstream: d5cff5df74b18e1e5ed94de8f4c9adee3ffd95c6 For the case of MIPS n64 target and 32-bit host, the computation of the DT_MIPS_RLD_MAP_REL tag involves sdyn->output_section->vma + sdyn->output_offset (64-bit) being added to b (32-bit host pointer), so losing the high part and resulting in an incorrect DT_MIPS_RLD_MAP_REL tag, and all dynamically linked glibc tests failing for n64. This patch fixes this (spot-tested with glibc tests; however, I don't have a self-contained testcase for this bug). * elfxx-mips.c (_bfd_mips_elf_finish_dynamic_sections) <DT_MIPS_RLD_MAP_REL>: Add target address to host address difference, not to host pointer. Change-Id: If4984d632723a36a3d6c739a96706b5636f975bc Conflicts: binutils-2.25/bfd/ChangeLog
| * Add support for DT_MIPS_RLD_MAP_REL.Matthew Fortune2016-04-0626-14/+178
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This tag makes it possible to access the debug map when debugging position independent executables. bfd/ * elfxx-mips.c (_bfd_mips_elf_create_dynamic_sections): Use executable instead of !shared to indicate an application vs shared library. (_bfd_mips_elf_size_dynamic_sections): Likewise. (_bfd_mips_elf_finish_dynamic_sections): Handle DT_MIPS_RLD_MAP_REL. (_bfd_mips_elf_get_target_dtag): Likewise. binutils/ * readelf.c (get_mips_dynamic_type): Handle DT_MIPS_RLD_MAP_REL. include/ * elf/mips.h (DT_MIPS_RLD_MAP_REL): New macro. ld/testsuite/ * ld-mips-elf/pic-and-nonpic-3b.ad: Adjust for extra dynamic tag. * ld-mips-elf/pic-and-nonpic-4b.ad: Likewise. * ld-mips-elf/pic-and-nonpic-5b.ad: Likewise. * ld-mips-elf/pic-and-nonpic-6-n32.ad: Likewise. * ld-mips-elf/pic-and-nonpic-6-n64.ad: Likewise. * ld-mips-elf/pic-and-nonpic-6-o32.ad: Likewise. * ld-mips-elf/tlsdyn-o32-1.d: Likewise. * ld-mips-elf/tlsdyn-o32-1.got: Likewise. * ld-mips-elf/tlsdyn-o32-2.d: Likewise. * ld-mips-elf/tlsdyn-o32-2.got: Likewise. * ld-mips-elf/tlsdyn-o32-3.d: Likewise. * ld-mips-elf/tlsdyn-o32-3.got: Likewise. * ld-mips-elf/tlsdyn-o32.d: Likewise. * ld-mips-elf/tlsdyn-o32.got: Likewise. * ld-mips-elf/pie-n32.d: New file. * ld-mips-elf/pie-n64.d: Likewise. * ld-mips-elf/pie-o32.d: Likewise. * ld-mips-elf/pie.s: Likewise. * ld-mips-elf/mips-elf.exp: Add new tests. Change-Id: I9fa38c2876a39c29ae3718bae0ade55c677f96fd
| * Revert "Add support for the new tag DT_MIPS_RLD_MAP2."Nikola Veljkovic2016-04-063-34/+3
| | | | | | | | This reverts commit 51d71fd2cc99f309cb792cc48ae910baeb92b2a0.
| * Revert "Fix DT_MIPS_RLD_MAP_REL tag for n64 target and 32-bit host."Nikola Veljkovic2016-04-062-7/+1
| | | | | | | | | | | | | | | | | | This reverts commit 7e80b87f7ff08c7c13f38d76a953b0cd753403aa. Conflicts: binutils-2.25/bfd/ChangeLog Change-Id: I1c0e3d5e9483001db6a4ffe807d6246c2f3ca126
| * Backport upstream CL to fix x86_32 clang segfaults.Yunlian Jiang2016-04-012-47/+78
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Upstream patch being backported: commit 84d543b7ed93bf6511cdf45791f4f0b717dfb718 Author: Sriraman Tallam <tmsriram@google.com> Date: Fri Feb 5 15:07:45 2016 -0800 2016-02-05 Sriraman Tallam <tmsriram@google.com> * icf.cc (get_rel_addend): New function. (get_section_contents): Move merge section addend computation to a new function. Ignore negative values for SHT_REL and SHT_RELA addends. Fix bug to not read past the length of the section. Fix bug related to addend computation for MERGE sections. Tested via cbuildbot for ChromeOS and build on Android mnc-dr-release for hammerhead
| * Backport upstream CL to fix x86_64 clang test-stlport segfaults.Han Shen2016-03-113-5/+47
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Upstream patch being backported: commit fc5a9bd57cbb974b8fc3aeb9a15d644cd9103451 Author: Cary Coutant <ccoutant@gmail.com> Date: Fri Feb 26 07:50:15 2016 -0800 Discard FDEs for zero-length address ranges. 2016-02-26 Egor Kochetov <egor.kochetov@intel.com> Cary Coutant <ccoutant@gmail.com> gold/ PR gold/19735 * ehframe.h (Cie::fde_encoding): New method. * ehframe.cc (Eh_frame::read_fde): Discard FDEs for zero-length address ranges. TESTED=only tested building with 'build.py' BUG=26085687 Change-Id: I52e90fba86b113a557dd4d29d36ecb4c512f3f7a
| * Silence "Erratum 843419 found and fixed" info messages from gold.Rahul Chaudhry2016-02-162-4/+5
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | BUG: 26536732 Tested by building binutils on ubuntu, and using the newly built gold to link volantis binaries. The info messages are gone. This patch is backported from upstream - commit e3dbf58233db3ad72f1ba624dc7f13ffd2728f25 Author: Han Shen <shenhan@google.com> Date: Thu Feb 11 09:47:49 2016 -0800 Remove info message for every erratum 843419 found and fixed. 2016-02-11 Rahul Chaudhry <rahulchaudhry@google.com> * aarch64.cc (Target_aarch64::scan_erratum_843419_span): Remove info message for every erratum 843419 found and fixed. Change-Id: I9792795096c919b006e98fdfdb01b448e597d605
| * Backport upstream patch to fix linker internal error.Han Shen2016-02-053-21/+127
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The error happens while linking a huge arm32 binary (for example, a debuggable chrome browser on arm) - the plt jump offset overflows. This fixes the problem by introducing another level of indirection. By default this does not affect anything. However in case of plt jump offset overflow, instead of triggering an internal error, the linker generates a proper message and suggests add "--long-plt" to the linker command. Tested via linking debug chrome browser for arm32. Also tested by build.py on android. The upstream patch - commit ce3e49806d505721e0875e704de0b6fcba7660ed Author: Peter Collingbourne <pcc@google.com> Date: Thu Dec 17 16:50:35 2015 -0800 Implement --long-plt flag (ARM only). gold/ PR gold/18780 * arm.cc (Target_arm::do_make_data_plt): Choose PLT generator based on value of --long-plt flag. (Output_data_plt_arm_standard::do_get_plt_entry_size): Moved to Output_data_plt_arm_short. (Output_data_plt_arm_standard::do_fill_plt_entry): Likewise. (Output_data_plt_arm_standard::plt_entry): Likewise. (Output_data_plt_arm_standard::do_fill_first_plt_entry): Fix variable reference. (Output_data_plt_arm_short): New class. (Output_data_plt_arm_short::do_fill_plt_entry): Error out on too large PLT offsets instead of asserting. (Output_data_plt_arm_long): New class. * options.h (General_options): Define --long-plt flag. Change-Id: Ia61126a09f2213d1ca5c3635ec9e5b36a63f6cf3
| * Backport trunk patch wrt wrong stub generated for aarch64 large DSOs.Han Shen2016-01-212-2/+11
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Tested by build.py for linux aarch64 toolchains. The upstream patch is: commit 9a472eda40ba686e45bf4922455518ffa3c887e1 Author: Han Shen <shenhan@google.com> Date: Fri Jan 15 09:31:23 2016 -0800 [gold][aarch64] PR gold/19472 - DSOs need pc-relative stubs. The stub generated during relaxation uses absolute addressing mode for shared libraries, which is not correct. Use pc-relative addressing instead. gold/ChangeLog: 2016-01-15 Han Shen <shenhan@google.com> PR gold/19472 - DSOs need pc-relative stubs. * aarch64.cc (Reloc_stub::stub_type_for_reloc): Return PC-relative stub type for DSOs and pie executables. Change-Id: Id0022975cf93600117ff5bf300b9d736d6ad1f80
| * Backport trunk patch for not applying dynamic relocs.Han Shen2016-01-153-1/+40
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Note, there is a small change from the origin patch - while in the upstream patch, the option --no-apply-dynamic is default off, it is on by default in this CL. This should not bring any changes to android or chromeos, because the relocation value this CL changes is not used by chromeos, and is not used by android either (after the fix in bionic months ago). The backported patch is - commit 0eccf19f96d6218dd7c2f8d13f0546c2b942cc08 Author: Cary Coutant <ccoutant@gmail.com> Date: Thu Nov 5 12:24:14 2015 -0800 Add aarch64-specific --no-apply-dynamic-relocs option. With --no-apply-dynamic-relocs on aarch64 targets, gold will not apply link-time values for absolute relocations that become dynamic relocations. This provides a workaround for broken Android dynamic linkers that use the link-time value as an extra addend to the relocation. gold/ PR gold/19163 * aarch64.cc (Target_aarch64::Relocate::relocate): Don't apply certain relocations if --no-apply-dynamic-relocs is set. * options.h (--apply-dynamic-relocs): New aarch64-specific option. Tested - build whole toolchain and built android N9 image. Change-Id: Ida524bbee2a83574cd9e78ffdad9270787327c78
* | Merge branch 'master' of https://android.googlesource.com/toolchain/binutils ↵Wolfgang Wiedmeyer2016-01-141-2/+12
|\ \ | |/ | | | | into HEAD
| * Remove internal link in warning messages.Han Shen2016-01-121-2/+12
| | | | | | | | | | | | | | | | | | The go/unsafe-pie link should not appear in any warning messages, replace it with a little bit detail. Tested by building locally on ubuntu. Change-Id: I3e590108ec2c6c8917475b5d78cdabbfcf2833f7
* | fix build error with gold aarch64Wolfgang Wiedmeyer2016-01-071-1/+1
|/ | | | | | original from here: https://www.sourceware.org/ml/binutils/2015-07/msg00234.html Signed-off-by: Wolfgang Wiedmeyer <wolfgit@wiedmeyer.de>
* Make genscripts.sh neutral to ChromeOS and Android.Han Shen2015-12-171-2/+28
| | | | | | | | | | | | | | ChromeOS reported a bug complaining about no default library search path after unification. Bug here - https://code.google.com/p/chromium/issues/detail?id=569605 Fix it for ChromeOS. This fix does not change anything for Android. Test: ChromeOS: manually test bug is fixed Android: ./build.py --host=linux --toolchain=x86_64 Change-Id: Ia900d081980d408c62a737917708e19842357e8e
* Merge "Fix darwin build failures (because of compilation warnings)."Han Shen2015-11-212-6/+16
|\
| * Fix darwin build failures (because of compilation warnings).Han Shen2015-11-212-6/+16
| | | | | | | | | | | | | | | | | | | | | | | | | | | | "%ld" is the printf specifier for "long int", which is 64-bit on 64-bit machines, and 32-bit on 32-bit hosts. While uint64_t is always 64 bits, so replace it with macro "PRIu64". Tested: ./build.py --host "linux" --toolchain aarch64-linux-android ./build.py --host "linux" --toolchain arm-linux-androideabi ./build.py --host "linux" --toolchain mipsel-linux-android ./build.py --host "windows" --toolchain arm-linux-androideabi Change-Id: I9f0b322ed14e6feb7a110bfebcff963cf94d7703
* | Cherry pick upstream patch for aarch64 unsupported reloc.Han Shen2015-11-212-0/+30
|/ | | | | | | | | | | | | | | | | | | | | Bug entry here - https://b.corp.google.com/u/0/issues/25642296 Tested: ./build.py --host "linux" --toolchain aarch64-linux-android Upstream patch here - commit 4d2f5d5824be99326c2c3e459c298e338856d8c8 Author: Han Shen <shenhan@google.com> Date: Tue Oct 27 15:17:54 2015 -0700 Patch for PR gold/19042 - unsupported reloc 311/312. gold/ * aarch64.cc (Target_aarch64::Scan::local): Add support for reloc 311/312. Change-Id: If784460c1ebdfb2be1ebdf1486db5eb03c5240a5
* Create an unified binutils source tree for both Android and ChromiumOS.Han Shen2015-11-20639-11703/+16353
| | | | | | | | | | | | | | | | | | | | | | About source code - The base version of this binutils is newer than that of aosp/binutils-2.25, it is based on the binutils that is used to build google products and ChromiumOS. And it contains *all* local Android patches as well as all patches that are cherry-picked from upstream for aosp/binutils-2.25 tree (up to Nov. 5 - 932d71b85). You may find the detailed development history for this binutils tree here - https://chromium.googlesource.com/chromiumos/third_party/binutils/+log/unification (This CL is a combination of all the CLs in it. After this CL is submitted the tree will be identical to https://chromium.googlesource.com/chromiumos/third_party/binutils/+log/unification at 2865a3615d80bd5f82d14d7e0484e84dc052596a) About testing - We tested this binutils for both ChromiumOS and Android. For android, we tested building N4, N5X, N6, N7, N9 using new binutils, we also did a full-build of toolchain (by build.py) and built a N5X image; for ChromiumOS - it passed ChromiumOS toolchain release tests on all 4 platforms (x86, x86_64, arm32 and arm64). Change-Id: I2bb2cf579f9458d0a8bc9612331dc7d5043e3d82
* Plumb --pic-veneer option for gold.Lee Campbell2015-11-053-3/+15
| | | | | | | | | | Cherry-pick of upstream commit cdb061674c044636465e52a8c989b9011f2ec8ac gold/ChangeLog: * arm.cc (Reloc_stub::stub_type_for_reloc): Plumb to stub generation. * options.h (General_options): Add --pic-veneer option. BUG: 25512672
* Fix Mac OS X build failures (wcsncasecmp unsat)Nick Clifton2015-11-035-6/+40
| | | | | | | | | | | | | | | | | Cherry-pick of upstream commit 31593e1b96c792abba3c5268d6423975aefa56b2 2015-02-24 Nick Clifton <nickc@redhat.com> * configure.ac (AC_CHECK_HEADERS): Add wctype.h. * configure: Regenerate. * config.in: Regenerate. * peXXigen.c: Include wctype.h if HAVE_WCTYPE_H is defined. (u16_mbtowc): Use wint_t types if HAVE_WCTYPE_H is defined. (rsrc_cmp): Use towlower instead of wcsncasecmp if HAVE_WCTYPE_H is defined. Bug: 25427405 Change-Id: I06e99d906226ce9c88ff1c568ae7a4d84df7f332
* Fix gold build error at -O0.Than McIntosh2015-10-162-2/+13
| | | | | | | | | | | | | | | | | | | | | Cherrypick of upstream commit 5d7908e0880030628536a0266968a15922574735 Author: Cary Coutant <ccoutant@gmail.com> Date: Thu Jun 11 13:19:38 2015 -0700 Fix gold build error at -O0. In aarch64.cc, Erratum_stub::STUB_ADDR_ALIGN and Reloc_stub::STUB_ADDR_ALIGN are declared as static const int, and initialized in the class body. These values are referenced from outside the class body, and with no optimization, the references go unresolved. gold/ * aarch64.cc (Erratum_stub::STUB_ADDR_ALIGN): Move initialization outside class body. (Reloc_stub::STUB_ADDR_ALIGN): Likewise. Change-Id: I6c0726e624be076e0d0565652a56da5ef761d88a
* Use "gold_info" instead of "gold_warning" for erratum fix.Han Shen2015-09-292-2/+10
| | | | | | | | | gold/ChangeLog: * aarch64.cc (Target_aarch64::scan_erratum_843419_span): Use 'gold_info'. (Target_aarch64::scan_erratum_835769_span): USe 'gold_info'. (cherry picked from commit 73854cdd438dc8858290231ebcb682377d71fec2)
* Drop 'missing mapping symbols' warning for arm and aarch64.Han Shen2015-09-293-13/+10
| | | | | | | | | | | | | | | | | | | | | | | | | The psABI (AAELF) says this about mapping symbols: 4.6.5.1 Section-relative mapping symbols Mapping symbols defined in a section define a sequence of half-open address intervals that cover the address range of the section. Each interval starts at the address defined by the mapping symbol, and continues up to, but not including, the address defined by the next (in address order) mapping symbol or the end of the section. A section must have a mapping symbol defined at the beginning of the section; however, if the section contains only data then the mapping symbol may be omitted. That makes it pretty clear that, in the absence of a mapping symbol, the entire section can be treated as data. gold/ChangeLog: * aarch64.cc (AArch64_relobj::scan_errata): Drop missing symbol warning. * arm.cc (Arm_relobj::scan_section_for_cortex_a8_erratum): Drop missing symbol warning. (cherry picked from commit 61163dfaaedb90d78e11c1d79ad1e469daf3da47)
* Remove duplicated option fix_cortex_a53_835769Andrew Hsieh2015-07-301-3/+0
| | | | | | | | | This was added before ld.gold scans for erratum 835769, but GCC has been hard-wired to pass "--fix-cortex-a53-835769" to linker (ld.bfd is default for AArch64 for now) unconditionally in Android platform. Change-Id: I39bacdc2a52d09afdc0993c17b46ac438d1fa246
* Make gold aarch64 accept long form of mapping symbols.Han Shen2015-07-292-3/+17
| | | | | | | | | | | | 2015-07-07 Han Shen <shenhan@google.com> gold/ChangeLog: 2015-07-06 Han Shen <shenhan@google.com> * aarch64.cc (AArch64_relobj::do_count_local_symbols): Make legal of mapping symbols. Change-Id: If62f577eeb704d8c7b0b2f525e91b3cb311001ca (cherry picked from commit b91deca94e36b9f18156909e4c615fe7af8e3ae8)
* Patch for gold internal error while fixing erratum 843419.Han Shen2015-07-292-4/+56
| | | | | | | | | | | | | | | | | | | The crash reason is that the insn to be moved to stub may be a relocation spot, so instead of placing the origin insn (that is insn before-relocation) to the stub, I have to place the relocated one. Note the relocation involved is non-pc-relative, so it is safe to move the relocated insn. gold/ChangeLog: 2015-06-29 Han Shen <shenhan@google.com> * AArch64.cc (Erratum_stub::Insn_utilities): New typedef. (Erratum_stub::update_erratum_insn): New method. (Stub_table::relocate_stubs): Modified to place relocated insn. (AArch64_relobj::fix_errata): Modified gold_assert. Change-Id: I9d90ac6e569bbbab086f877c7446dc6f9ea96247 (cherry picked from commit 56b06706ebe6bb3decc674b788b6f5b243b8610d)
* [gold][aarch64] Fix erratum 835769.Han Shen2015-07-293-44/+285
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | gold/ChangeLog: * aarch64.cc (AArch64_insn_utilities::BYTES_PER_INSN): Move defintion outside class definition. (AArch64_insn_utilities::AARCH64_ZR): New static constant. (AArch64_insn_utilities::aarch64_op31): New member. (AArch64_insn_utilities::aarch64_ra): New member. (AArch64_insn_utilities::aarch64_mac): New member. (AArch64_insn_utilities::aarch64_mlxl): New member. (ST_E_835769): New global enum member. (Stub_table::relocate_stubs): Add 835769 handler. (Stub_template_repertoire::Stub_template_repertoire): Install new stub type. (AArch64_relobj::scan_errata): This func is renamed from scan_erratum_843419. (AArch64_relobj::do_count_local_symbols): Add 835769 handler. (AArch64_relobj::do_relocate_sections): Add 835769 handler. (AArch64_relobj::scan_sections_for_stubs): Add 835769 handler. (Target_aarch64::scan_erratum_835769_span): New method. (Target_aarch64::create_erratum_stub): New method. (Target_aarch64::is_erratum_835769_sequence): New method. (Target_aarch64::scan_erratum_843419_sequence): Move part of the code into create_erratum_stub. * options.h (fix_cortex_a53_835769): New option. Change-Id: Ia355fad38de61f82b9a42c601129fbbe2730ba27 (cherry picked from commit 2f0c79aa61e82cbc5da4b9ab399362b2cdd8ea2e)
* Patch for erratum-843419 (2 of 2 - fix erratum occurrences).Han Shen2015-07-292-176/+710
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Now fixing for 843419 is fully functional. The first part of the erratum fix CL is here - https://sourceware.org/ml/binutils/2015-04/msg00229.html gold/ChangeLog 2015-06-10 Han Shen <shenhan@google.com> * aarch64.cc(global enum): New constants representing stub types. (Stub_template): New POD struct. (Stub_template_repertoire): New class. (Stub_base): New class. (Erratum_stub): New class. (Reloc_stub): Refactored to be a subclass of Stub_base. (Reloc_stub::Stub_type): Removed. (Reloc_stub::offset): Moved to Stub_base. (Reloc_stub::set_offset): Moved to Stub_base. (Reloc_stub::destination_address): Moved to Stub_base. (Reloc_stub::set_destination_address): Moved to Stub_base. (Reloc_stub::reset_destination_address): Moved to Stub_base. (Reloc_stub::stub_type): Renamed and moved to Stub_base. (Reloc_stub::stub_size): Renamed and moved to Stub_base. (Reloc_stub::stub_insns): Renamed and moved to Stub_base. (Reloc_stub::write): Moved to Stub_base. (Reloc_stub::invalid_offset): Moved to Stub_base. (Reloc_stub::invalid_address): Moved to Stub_base. (Reloc_stub::stub_type_): Renamed and moved to Stub_base. (Reloc_stub::stub_insns_): Moved to Stub_base. (Reloc_stub::offset_): Moved to Stub_base. (Reloc_stub::destination_address_): Moved to Stub_base. (Stub_table::The_aarch64_relobj): New typedef. (Stub_table::The_erratum_stub): New typedef. (Stub_table::The_erratum_stub_less): New typedef. (Stub_table::The_erratum_stub_set): New typedef. (Stub_table::The_erratum_stub_set_iter): New typedef. (Stub_table::empty): Added emptiness testing for erratum stubs. (Stub_table::add_erratum_stub): New method to add an erratum stub. (Stub_table::find_erratum_stub): New method. (Stub_table::find_erratum_stubs_for_input_section): New method. (Stub_table::erratum_stub_address): New method. (Stub_table::update_date_size_changed_p): Modified to handle erratum stubs. (Stub_table::do_addralign): Modified to handle erratum stubs. (Stub_table::erratum_stubs_): New member. (Stub_table::erratum_stub_size_): New member. (Stub_table::relocate_stubs): Modified to handle erratum stubs. (Stub_table::do_write): Modified to handle erratum stubs. (AArch64_relobj::The_erratum_stub): New typedef. (AArch64_relobj::Erratum_stub_set_iter): New typedef. (AArch64_relobj::fix_errata): New method. (Target_aarch64::The_reloc_stub_type): Removed. (Target_aarch64::The_erratum_stub): New typede. (AArch64_relocate_functions::construct_b): New method. Change-Id: I37ccc9fbefd5559dae700637e119a2a885cf3eb9 (cherry picked from commit a48d0c12f893a922692492829f04189d9f216172)
* Fix DT_MIPS_RLD_MAP_REL tag for n64 target and 32-bit host.Joseph Myers2015-07-272-1/+7
| | | | | | | | | | | | | | | | | | Cherry-picked from upstream: d5cff5df74b18e1e5ed94de8f4c9adee3ffd95c6 For the case of MIPS n64 target and 32-bit host, the computation of the DT_MIPS_RLD_MAP_REL tag involves sdyn->output_section->vma + sdyn->output_offset (64-bit) being added to b (32-bit host pointer), so losing the high part and resulting in an incorrect DT_MIPS_RLD_MAP_REL tag, and all dynamically linked glibc tests failing for n64. This patch fixes this (spot-tested with glibc tests; however, I don't have a self-contained testcase for this bug). * elfxx-mips.c (_bfd_mips_elf_finish_dynamic_sections) <DT_MIPS_RLD_MAP_REL>: Add target address to host address difference, not to host pointer. Change-Id: If4984d632723a36a3d6c739a96706b5636f975bc
* Fix an internal error in do_print_to_mapfile on Arm.Cary Coutant2015-07-141-0/+5
| | | | | | | | | | | | | 2015-02-02 Khem Raj <raj.khem@gmail.com> gold/ * attributes.h (class Output_attributes_section_data ): Add do_print_to_mapfile function. BUG=21451238 b.android.com/177690 Change-Id: I56d853784a48c247578d08bca848869b3a4c2c78
* Apple has had wcsncasecmp since 10.7.Dan Albert2015-06-221-22/+0
| | | | | | | | AFAICT this was an Android hack. I don't see it anywhere in the history for upstream. Just remove it. Bug: http://b/21534147 Change-Id: I7c69f543a80a0e4805fbea2934eaec052544d6c7
* Disable the warning of a hidden symbol matches a reference to beYiran Wang2015-06-162-9/+6
| | | | | | | | | resolved. It looks like it is WAI, but in some scenario there could a lot of them and the build script just die on any warning. Please refer to bug 21502954 for some details. Change-Id: I6cc90adccc63d26f15821bed6e4c6ff336722eff
* Fix darwin buildAndrew Hsieh2015-05-062-3/+34
| | | | | | | | 1. In Drawin PTHREAD_ONCE_INIT is {0x30B1BCBA, {0}} and the GCC < 4.4 doesn't support ended initializer list 2. wcsncasecmp doesn't exist in MacSDK10.6.x Change-Id: I69204a72f853f5263dffedc448379d75ed4eca2e
* Merge "MIPS: Update the encoding of the d16mule MXU instruction."Andrew Hsieh2015-05-053-18/+18
|\
| * MIPS: Update the encoding of the d16mule MXU instruction.Andrew Bennett2015-04-223-18/+18
| | | | | | | | Change-Id: I7ea5748faea472beb79e34aee8d19c4e0f37981e
* | Rename '--fix-cortex-a53' to '--fix-cortex-a53-843419'.Han Shen2015-04-283-8/+13
| | | | | | | | | | | | | | | | | | | | | | | | | | | | Keep gold consistent with bfd erratum-fixing option names, so as to ease life in Makefile/scripts. gold/ * options.h (--fix-cortex-a53-843419): Rename option. * aarch64.cc (AArch64_relobj::do_count_local_symbols): Use renamed option. (AArch64_relobj::scan_sections_for_stubs): Use renamed option. Change-Id: I8941fb38ef34fa5eb297e83725d87fdee994fc3a
* | The first patch to address cortex-a53 erratum-843419.Han Shen2015-04-283-2/+600
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | It implemented scanning the binary and reporting occurrences to users when '--fix-cortex-a53' is turned on. With this, gold users will be able to see if or not there are such erratum occurrences in the output binary. Also included in the CL is reading/recording mapping symbols, which is needed during scan. gold/ChangeLog: * aarch64.cc (AArch64_insn_utilities): New utility class. (AArch64_relobj::Mapping_symbol_position): New struct. (AArch64_relobj::Mapping_symbol_info): New typedef. (AArch64_relobj::do_count_local_symbols): New function overriding parent's implementation. (AArch64_relobj::mapping_symbol_info_): New member (AArch64_relobj::scan_erratum_843419): New method. (Target_aarch64::scan_erratum_843419_span): New method. (Target_aarch64::is_erratum_843419_sequence): New method. * options.h (fix_cortex_a53): New option. Change-Id: I4fe7a2b8858cda358ac28eede42cd5a2f49c2238
* | [2.24, 2.25] Convert mov foo@GOT(%reg), %reg to lea foo@GOTOFF(%reg), %regAlexander Ivchenko2015-04-218-33/+235
|/ | | | | | | | | | | | | | | | | | | | | | 2015-04-01 Ilya Tocar <ilya.tocar@intel.com> PR gold/17640 * i386.cc (Target_i386::can_convert_mov_to_lea): New. (Target_i386::Scan::local): Don't create GOT entry, when we can convert GOT to GOTOFF. (Target_i386::Scan::global): Ditto. (Target_i386::Relocate::relocate): Convert mov foo@GOT(%reg), %reg to lea foo@GOTOFF(%reg), %reg if possible. * testsuite/Makefile.am (i386_mov_to_lea): New test. * testsuite/i386_mov_to_lea1.s: New. * testsuite/i386_mov_to_lea2.s: Ditto. * testsuite/i386_mov_to_lea3.s: Ditto. * testsuite/i386_mov_to_lea4.s: Ditto. * testsuite/i386_mov_to_lea5.s: Ditto. * testsuite/i386_mov_to_lea.sh: Ditto. Change-Id: I7916a1da20873600a910ce358990b1fe2c1f9897 Signed-off-by: Alexander Ivchenko <alexander.ivchenko@intel.com>