New issue
Advanced search Search tips

Issue 859658 link

Starred by 1 user
Status: Verified
Owner:
manojgupta@chromium.org
Closed: Jul 3
Components:
EstimatedDays: ----
NextAction: ----
OS: Chrome
Pri: 2
Type: Bug
Build-Toolchain



Sign in to add a comment

Stop using GCC in cros-i686 eclass

Project Member Reported by manojgupta@chromium.org, Jul 2 
cros-i686 eclass is using GCC as compiler for 32-bit builds.
Switch it to use clang instead.

So far seems like crash-reporter (32-bit) is the only package using cros-i686 eclass.
Project Member

Comment 1 by bugdroid1@chromium.org, Jul 3 

The following revision refers to this bug:
  https://chromium.googlesource.com/chromiumos/overlays/chromiumos-overlay/+/823317f722d0d616ddf88acb4b25c2736f79f58f

commit 823317f722d0d616ddf88acb4b25c2736f79f58f
Author: Manoj Gupta <manojgupta@google.com>
Date: Tue Jul 03 11:13:35 2018

cros-i686: Stop forcing GCC as compiler.

Should be fine to use clang as the compiler for i686.
Also pass "-stdlib=libstdc++" to linker for 32-bit builds.

BUG= chromium:859658 
TEST=eve-release passed.

Change-Id: Ie4e16734d7b9f2c630f640fa904cf372812c52bc
Reviewed-on: https://chromium-review.googlesource.com/1123227
Commit-Ready: Manoj Gupta <manojgupta@chromium.org>
Tested-by: Manoj Gupta <manojgupta@chromium.org>
Reviewed-by: Yunlian Jiang <yunlian@chromium.org>

[modify] https://crrev.com/823317f722d0d616ddf88acb4b25c2736f79f58f/eclass/cros-i686.eclass

Comment 2 by manojgupta@chromium.org, Jul 3

Status: Verified (was: Untriaged)
►

Sign in to add a comment