ICF doesn't respect section alignment |
|||||
Issue descriptiongold linker has the following bug. ChromeOS uses icf=all. https://sourceware.org/bugzilla/show_bug.cgi?id=17704 Do we need to disable icf? or backport a binutils patch? (is there one?)
,
Aug 24 2016
CL out for review: https://android-review.googlesource.com/#/c/262195/
,
Oct 21 2016
,
Nov 3 2016
CL merged: https://android-review.googlesource.com/#/c/292548/ Remaining steps: -) Update Chromium OS binutils ebuild to use the new commit/tree hashes. -) Update the toolchain prebuilts for Android.
,
Nov 12 2016
The following revision refers to this bug: https://chromium.googlesource.com/chromiumos/overlays/chromiumos-overlay/+/257c49a287a1954c3cabd80ad6ea5395b28dbb2c commit 257c49a287a1954c3cabd80ad6ea5395b28dbb2c Author: Rahul Chaudhry <rahulchaudhry@chromium.org> Date: Thu Nov 10 23:10:09 2016 sys-devel/binutils: sync to latest binutils sources from aosp. This CL syncs binutils in chromiumos to the latest from aosp. The commit from aosp/master being imported in this CL is: b8ff982 gold: make icf respect section alignment requirements. BUG= chromium:628795 TEST=`sudo emerge sys-devel/binutils` works. TEST=`sudo emerge cross-i686-pc-linux-gnu/binutils` works. TEST=`sudo emerge cross-x86_64-cros-linux-gnu/binutils` works. TEST=`sudo emerge cross-armv7a-cros-linux-gnueabi/binutils` works. TEST=`Tested as an epatch with https://chromium-review.googlesource.com/#/c/401891/` Change-Id: I24019b81f9b6e88d2657c97e72ad98c452b78844 Reviewed-on: https://chromium-review.googlesource.com/410440 Commit-Ready: Rahul Chaudhry <rahulchaudhry@chromium.org> Tested-by: Rahul Chaudhry <rahulchaudhry@chromium.org> Reviewed-by: Luis Lozano <llozano@chromium.org> [rename] https://crrev.com/257c49a287a1954c3cabd80ad6ea5395b28dbb2c/sys-devel/binutils/binutils-2.25.51-r62.ebuild
,
Nov 14 2016
,
Aug 3 2017
Closing. Please reopen it if its not fixed. Thanks! |
|||||
►
Sign in to add a comment |
|||||
Comment 1 by llozano@chromium.org
, Jul 18 2016