Closed Bug 774372 Opened 10 years ago Closed 10 years ago

build clang packages with --disable-assertions

Categories

(Release Engineering :: General, defect)

x86_64
Linux
defect
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: espindola, Assigned: espindola)

Details

Attachments

(2 files)

Clang builds (like gcc trunk but unlike gcc releases) defaults to having lots of sanity checks. This was good when we started, but it has been a long time since those asserts have found a bug when building firefox.

I timed a build on try:

With this patch and without ccache:
universal: 79m
64 bit debug: 55m
32 bit debug: 58m

Without this patch and without ccache:
universal: 86m
64 bit debug: 59m
32 bit debug: 61m

With gcc and ccache:
universal: 97m
64 bit debug: 65m
32 bit debug: 71m

So we should be able to switch to clang witt cache disable and then benchmark adding it back.
Attachment #642650 - Flags: review?(rail)
Assignee: nobody → respindola
Status: NEW → ASSIGNED
Attachment #642652 - Flags: review?(rail)
Attachment #642650 - Flags: review?(rail) → review+
Attachment #642652 - Flags: review?(rail) → review+
https://hg.mozilla.org/mozilla-central/rev/b3b880f37cd4
https://hg.mozilla.org/mozilla-central/rev/d685dfe6eb7b
Status: ASSIGNED → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
Product: mozilla.org → Release Engineering
You need to log in before you can comment on or make changes to this bug.