[Cronet] Use XCode Clang when building release version of Cronet for iOS |
|||||
Issue descriptionThe binary size of fat Cronet.framework is reduced if gn argument use_xcode_clang is true. Built using Chromium clang: 6415456 Nov 17 14:13 Cronet Built using XCode clang: 6381632 Nov 17 16:27 Cronet The difference is just 33k, but looks like an easy save.
,
Nov 18 2016
,
Jan 9 2017
Recent build with CookieStoreIOS shows even bigger difference (about 80k): Built using Chromium clang: 6747488 Jan 6 17:15 Cronet Built using XCode clang: 6664000 Jan 9 10:45 Cronet
,
May 11 2017
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/970e26717675d235f8dc59c1646e4abf9e3a7784 commit 970e26717675d235f8dc59c1646e4abf9e3a7784 Author: mef <mef@chromium.org> Date: Thu May 11 21:27:01 2017 [Cronet] Use xcode clang on iOS to reduce release binary size. BUG= 666502 CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.android:android_cronet_tester Review-Url: https://codereview.chromium.org/2614423002 Cr-Commit-Position: refs/heads/master@{#471077} [modify] https://crrev.com/970e26717675d235f8dc59c1646e4abf9e3a7784/components/cronet/tools/cr_cronet.py [modify] https://crrev.com/970e26717675d235f8dc59c1646e4abf9e3a7784/components/cronet/tools/package_ios.py
,
May 11 2017
Per conversation with Sleevi all official bots should use is_official_build=true, however this flag is not available for debug builds, so use_xcode_clang=true looks like a next good option. Sana, could you help us to update Cronet bots?
,
May 12 2017
The following revision refers to this bug: https://chrome-internal.googlesource.com/chrome/ios_internal.git/+/f21a682c327d6411e3df2c35e331c263b841f272 commit f21a682c327d6411e3df2c35e331c263b841f272 Author: smut <smut@google.com> Date: Fri May 12 22:04:29 2017
,
May 12 2017
|
|||||
►
Sign in to add a comment |
|||||
Comment 1 by jif@chromium.org
, Nov 18 2016Status: Assigned (was: Untriaged)