Project: skia Issues People Development process History Sign in
New issue
Advanced search Search tips
Note: Color blocks (like or ) mean that a user may not be available. Tooltip shows the reason.
Issue 2679 Add a Windows clang asan bot.
Starred by 6 users Project Member Reported by bungeman@google.com, Jun 19 2014 Back to list
Status: Accepted
Owner:
Cc:
Components:
Area: Infrastructure
Priority: Medium
Type: Defect

Blocked on:
issue 761



Sign in to add a comment
Chromium has an fyi bot at http://build.chromium.org/p/chromium.fyi/builders/Chromium%20Win%20Clang%20(asan) which builds with clang for Windows and runs asan. This is awesome, and Skia should have some of this awesome too. The clang on Windows build has discovered issues in Skia, so just having the clang builder would be good.


 
Comment 1 by epoger@google.com, Aug 18 2014
Cc: -epoger@google.com
Project Member Comment 2 by bore...@google.com, Aug 20 2014
Blockedon: skia:761
Project Member Comment 3 by tha...@chromium.org, Aug 25 2014
(Note that clang/win is pretty experimental – there's no support for exceptions yet, no goma support, plugins don't work, only linetable debug debug information, etc. But clang/win can fall back to cl.exe for things it can't compile – like exceptions – and the static library release fyi bot we have looks pretty good.

If you're happy with this not being fully production-quality yet, things should mostly work out. That's why the chromium bot is fyi-only at this point.)
Project Member Comment 4 by bore...@google.com, Mar 24 2015
Labels: Component-Buildbots
Project Member Comment 5 by bore...@google.com, Jan 19 2016
Labels: NewBotRequest
Project Member Comment 6 by bore...@google.com, Feb 9 2016
Also need a compile bot for Clang on Windows.
Project Member Comment 7 by tha...@chromium.org, Feb 9 2016
(a Windows clang asan bot would double as a compile bot for clang on windows)
Project Member Comment 8 by robertphillips@google.com, Mar 10 2016
Cc: rmis...@google.com jcgrego...@google.com bore...@google.com
 Issue 5078  has been merged into this issue.
Comment 9 Deleted
Project Member Comment 10 by bungeman@google.com, Mar 16
This isn't really just a nice to have at this point, it's something we should really have (at the very least the compile bot). The win_clang bot is now part of the Chromium commit queue and we have no advance warning when it's going to fail. Also, it's been nearly three years since this bug was filed and clang for Windows has advanced quite a bit, it builds all of Chromium and Microsoft is shipping the front end. I would prefer not to use the front end Microsoft ships since that's not what Chromium is using, but even that would be an improvement.

I'll take a quick look at the build side of this when I get the chance.
Project Member Comment 11 by bore...@google.com, Mar 16
Sorry for my lack of response here.  If you can give me a series of commands to perform the build, I can add the bot.
Project Member Comment 12 by tha...@chromium.org, Mar 16
For completeness, all the stuff mentioned in comment 3 works now.

Do you have bots that use chromium's clang? If so, just do that (chromium's deps file downloads clang on win by default these days, and is_clang=true in your args.gn selects that as default compiler. If you want asan, add is_asan=true). If you don't deps in chromium's tools/clang/scripts and build/, I'd need some information on your setup to advise.
Sign in to add a comment