Open Bug 1404433 Opened 2 years ago Updated 19 days ago

Get coverage build with Clang on Linux green

Categories

(Testing :: Code Coverage, enhancement)

Version 3
enhancement
Not set

Tracking

(Not tracked)

People

(Reporter: marco, Unassigned, NeedInfo)

References

(Blocks 1 open bug)

Details

There are quite a few failures, e.g. see https://treeherder.mozilla.org/#/jobs?repo=try&revision=e30d8c3c59a9737f1825e81e6901774f562ede00 or https://treeherder.mozilla.org/#/jobs?repo=try&revision=014e361716000d0dee4427bf7195316a2aba7b21.

Most of them are timeouts, some of them are due to leak checking (that we could disable like we do for asan).

Greg, do you think you could help here? You have a lot of experience as you've done the same with the GCC build too!
Sure thing! I'll get to it within the next couple weeks. I'm leaving a ni? open here for myself.
Flags: needinfo?(gmierz2)
Depends on: 1414962
Here's a test run for the latest changes (including the clang patches): https://treeherder.mozilla.org/#/jobs?repo=try&revision=73dc9ddd7ee0e3b72c0d80bbe67b7a462760de16
So, the build that uses the RUSTFLAG option is failing. But the regular clang build without it is working. There are many timeouts though and only a handful of bugs from the looks of it, which is promising.
(In reply to Greg Mierzwinski [:sparky] from comment #4)
> So, the build that uses the RUSTFLAG option is failing. But the regular
> clang build without it is working. There are many timeouts though and only a
> handful of bugs from the looks of it, which is promising.

Strange, the build with the Rust flags is failing because it doesn't recognize the "profile" option ([task 2017-12-16T05:14:18.647Z] 05:14:18     INFO -  error: unknown codegen option: `profile`). So it seems it is not actually using the custom Rust build.
Here's a linux64 clang coverage run without any task timeouts, only bugs to fix: https://treeherder.mozilla.org/#/jobs?repo=try&revision=f60c1e4bbf038e3333ca22e79fc667fbebd869f8

Should we file bugs for each of them?
(In reply to Greg Mierzwinski [:sparky] from comment #6)
> Here's a linux64 clang coverage run without any task timeouts, only bugs to
> fix:
> https://treeherder.mozilla.org/#/
> jobs?repo=try&revision=f60c1e4bbf038e3333ca22e79fc667fbebd869f8
> 
> Should we file bugs for each of them?

One failure is due to a wrong year (2017 instead of 2018), that's an interesting one :)
Most of the failures in web-platform tests are timeouts, we could try increasing the timeout multiplier like I did in https://bugzilla.mozilla.org/page.cgi?id=splinter.html&bug=1418283&attachment=8933264 (I increased it to 4, maybe we can increase it a little more).
Also, we need to add the "--llvm" parameter to the gcov invocation.

With these changes, could you try respinning another try build and re-running the failures 5 times (so we make sure they are not intermittents)?
Blocks: 1471339
No longer blocks: 1335518
No longer blocks: 1471339
Blocks: 1499663
Depends on: 1599436
You need to log in before you can comment on or make changes to this bug.