New issue
Advanced search Search tips

Issue 656044 link

Starred by 3 users

Issue metadata

Status: Duplicate
Merged: issue 481675
Owner: ----
Closed: Jun 2017
EstimatedDays: ----
NextAction: ----
OS: Linux
Pri: 3
Type: Bug

Blocking:
issue 481675



Sign in to add a comment

Make chrome/android built with clang by default, x86

Project Member Reported by thakis@chromium.org, Oct 14 2016

Issue description

Current size difference:

target_cpu = "x86"
is_clang = false
thakis@thakis:~/src/chrome/src$ ls -hl out/gnand/gen/chrome/android/chrome_public_apk__prepare_native/packed-libs/libchrome.so
-rwxr-x--- 1 thakis eng 65M Oct 10 15:58 out/gnand/gen/chrome/android/chrome_public_apk__prepare_native/packed-libs/libchrome.so
is_clang = true
thakis@thakis:~/src/chrome/src$ ls -hl out/gnand2/gen/chrome/android/chrome_public_apk__prepare_native/packed-libs/libchrome.so
-rwxr-x--- 1 thakis eng 73M Oct 10 16:20 out/gnand2/gen/chrome/android/chrome_public_apk__prepare_native/packed-libs/libchrome.so


Without -mstackrealign:

thakis@thakis:~/src/chrome/src$  ls -hl out/gnand/gen/chrome/android/chrome_public_apk__prepare_native/packed-libs/libchrome.so
-rwxr-x--- 1 thakis eng 65M Oct 14 12:14 out/gnand/gen/chrome/android/chrome_public_apk__prepare_native/packed-libs/libchrome.so
thakis@thakis:~/src/chrome/src$  ls -hl out/gnand2/gen/chrome/android/chrome_public_apk__prepare_native/packed-libs/libchrome.so
-rwxr-x--- 1 thakis eng 71M Oct 14 12:35 out/gnand2/gen/chrome/android/chrome_public_apk__prepare_native/packed-libs/libchrome.so

(first is gcc, second is clang)

We need to figure out where the size goes.
 

Comment 1 by thakis@chromium.org, Oct 14 2016

Summary: Make chrome/android built with clang by default, x86 (was: Make chrome/android built with clang, x86)

Comment 2 by thakis@chromium.org, Oct 14 2016

Blockedon: 656046

Comment 3 by thakis@chromium.org, Oct 14 2016

Blockedon: -656046

Comment 4 by thakis@chromium.org, Jun 14 2017

Mergedinto: 481675
Status: Duplicate (was: Available)
We ended up switching all archs at once after all.

Sign in to add a comment