New issue
Advanced search Search tips
Note: Color blocks (like or ) mean that a user may not be available. Tooltip shows the reason.

Issue 661893 link

Starred by 1 user

Issue metadata

Status: Verified
Owner:
Last visit > 30 days ago
Closed: Mar 2017
Cc:
EstimatedDays: ----
NextAction: ----
OS: Chrome
Pri: 2
Type: Bug

Blocking:
issue 537368



Sign in to add a comment

moblab: use LLVM to build moblab

Project Member Reported by llozano@chromium.org, Nov 3 2016

Issue description

We are now using LLVM to build most of ChromeOS for AMD64 boards.
When we did the transition we could not test moblab because it was broken.
But, moreover, we think it is better if you test this.

To test, please revert the following CL and build moblab. It should build with LLVM now. 

https://chromium-review.googlesource.com/397863

Please do this soon. We don't want to keep boards building with GCC for long.
Note, that just building is not a good enough test. There is a small chance of runtime bugs.
 

Comment 1 by sbasi@chromium.org, Nov 4 2016

Cc: ntang@chromium.org krk@chromium.org
I'll do this today but just fyi I've moved off MobLab for the most part.

Comment 2 by sbasi@chromium.org, Nov 4 2016

Created revert CL: https://chromium-review.googlesource.com/#/c/407879/

Kicked off guado_moblab-release build (would paladin be okay cause thats faster?):
https://uberchromegw.corp.google.com/i/chromiumos.tryserver/waterfall?committer=sbasi@google.com&builder=release
paladin is not ok since it does not rebuild everything...

Comment 4 by krk@chromium.org, Nov 4 2016

Cc: cros-peng-moblab@google.com

Comment 5 by lloz...@google.com, Mar 2 2017

can this be closed? 
sbasi@?

Comment 6 by sbasi@chromium.org, Mar 6 2017

Status: Fixed (was: Assigned)

Comment 7 by dchan@google.com, May 30 2017

Labels: VerifyIn-60

Comment 8 by dchan@chromium.org, Aug 1 2017

Labels: VerifyIn-61
Status: Verified (was: Fixed)
Closing. Please reopen it if its not fixed. Thanks!

Sign in to add a comment