695726 - Build Firefox nightlies using clang

Status: RESOLVED DUPLICATE of bug 672210

(Infrastructure & Operations Graveyard :: CIDuty, task, P3)

Description

[Rail Aliiev [:rail]]

Platforms: linux, linux64, macosx64

Comment 1

[Rail Aliiev [:rail]]

I think you both would be interested in this. :)

Comment 2

[Rafael Ávila de Espíndola (:espindola) (not reading bugmail)]

cool. macosx64 == 64 bit debug or universal?

Comment 3

[Rail Aliiev [:rail]]

(In reply to Rafael Ávila de Espíndola (:espindola) from comment #2)
> cool. macosx64 == 64 bit debug or universal?

I think it should be universal for opt build and 32 and 62 bit for debug. Have no strong opinion here.

Comment 4

[Rail Aliiev [:rail]]

I tried to compile m-c on linux with latest installed clang (bug 693605). I changed nightly mozconfig to have this:

CC="/tools/clang-3.0/bin/clang -fgnu89-inline"
CXX=/tools/clang-3.0/bin/clang++

The attempt failed with:
================
clang: warning: argument unused during compilation: '-I ecl'
mpi/mpi_x86.s:65:1: error: Encountered a .else that doesn't follow a .if or an .elseif
.else
^
mpi/mpi_x86.s:66:1: error: macro 'GET' is already defined
.macro GET   var,reg
^
mpi/mpi_x86.s:69:1: error: macro 'PUT' is already defined
.macro PUT   reg,var
^
mpi/mpi_x86.s:72:1: error: Encountered a .endif that doesn't follow a .if or .else
.endif
^
mpi/mpi_x86.s:577:1: error: unmatched .ifs or .elses

^
make[8]: *** [/builds/slave/m-cen-lnx-clang-ntly/build/obj-firefox/nss/freebl/Linux_SINGLE_SHLIB/mpi_x86.o] Error 1
================

Comment 5

[Rafael Ávila de Espíndola (:espindola) (not reading bugmail)]

(In reply to Rail Aliiev [:rail] from comment #3)
> (In reply to Rafael Ávila de Espíndola (:espindola) from comment #2)
> > cool. macosx64 == 64 bit debug or universal?
> 
> I think it should be universal for opt build and 32 and 62 bit for debug.
> Have no strong opinion here.

If we have enough resources, it would be really nice to do every build we do with gcc with clang too. If not, I guess the priorities should be

* os x 32 bit debug
* os x 64 bit debug
* os x universal nightly
* linux32 debug
* linux64 debug
* linux32 opt
* linux64 opt

The preference for os X is because we should switch to clang at some point in it. The preference for debug is because more code is compiled in debug mode.

Comment 6

[Rafael Ávila de Espíndola (:espindola) (not reading bugmail)]

(In reply to Rail Aliiev [:rail] from comment #4)
> I tried to compile m-c on linux with latest installed clang (bug 693605). I
> changed nightly mozconfig to have this:

> clang: warning: argument unused during compilation: '-I ecl'
> mpi/mpi_x86.s:65:1: error: Encountered a .else that doesn't follow a .if or
> an .elseif
> .else


This was a bug in llvm:

http://llvm.org/viewvc/llvm-project?view=rev&revision=142537

:-(

Comment 7

[Rafael Ávila de Espíndola (:espindola) (not reading bugmail)]

A try run in
https://tbpl.mozilla.org/?tree=Try&rev=fe13a5c6469f

shows that we can already do OS X 64 bits, both opt and debug. We should probably do those first.

Comment 8

[Rail Aliiev [:rail]]

Unfortunately I have no time to touch this until mid February. Back to pool.

The main idea was adding new platforms (adding a new toolkit for the existing platforms aren't trivial :/ ). The initial patch can be find here: https://gist.github.com/1621526. Mozconfig part could be ignored. The patch adds new platforms which have nightlies only.

Comment 9

[John O'Duinn [:joduinn] (please use "needinfo?" flag)]

found in triage. 

There's a few bugs tracking switching compilers to clang. We're going to use bug#672210 to track that work.