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

Issue 656052 link

Starred by 2 users

Issue metadata

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

Blocking:
issue 481675



Sign in to add a comment

Make chrome/android built with clang by default, mips32 and mips64

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

Issue description

We want to switch chrome/android to build with clang by default. We want to use the autoupdating compiler stuff we use everywhere.

To make sure that upstream clang compiler changes don't break chrome, we have bots building and testing chrome with clang trunk (https://build.chromium.org/p/chromium.fyi/console?category=clang%20tot). We don't have any mips devices that we run tests on (either on that waterfall or anywhere else). I suppose we can add a bot that at least compiles, which gives some coverage.

I don't know how binary size or perf compares to gcc on mips. I don't know if any mips folks want to do any evaluation before we switch. See other blockers of linked bug for other platforms.
 

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

Summary: Make chrome/android built with clang by default, mips32 and mips64 (was: Make chrome/android built with clang by default, mips32)
Actually, let's use this bug for mips32 and mips64 for now, if this turns out to be a bigger project, we can split the bug.
There are some ongoing build issues for MSA built-ins with clang 4.0.0
(Available at chromium/src/third_party/llvm-build/Release+Asserts/bin/clang)
And i believe upstream of LLVM fixes is also in progress.

There seems no issues in adding mips32/mips64 bots (disabling msa) as of now as those builds fine. We can start with mips32/64 builds and enable msa once clang build issues gets fixed.

For running tests, is it possible to setup QEMU for linux from ?
http://codescape-mips-sdk.imgtec.com/components/qemu/2.5.0.2.0/

Thanks,

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

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

Sign in to add a comment