Should we keep rnk's local patch in this roll?

I think we should drop the instcombine patch for now. If we want it, let's get it upstream under an -mllvm flag.

Agreed.

I'd be cool if we could wait for my -Wdelete-non-virtual-dtor cl too, should make it in today.

r312167 for  crbug.com/673171 

r312178 for  crbug.com/760725 

r312182 for  crbug.com/760768 

The bots are pretty red, but I think most of the issues are accounted for, and the Win redness is due to the  crbug.com/673171  work.

Started packaging here, but I probably won't be around to see it finish tonight: https://chromium-review.googlesource.com/c/chromium/src/+/644095

Tryjobs: https://chromium-review.googlesource.com/c/chromium/src/+/644095

Can we get r312583 in this roll?  It should fix the PDB size issue.

Trying 312679: https://chromium-review.googlesource.com/c/chromium/src/+/656062

Goma is ready; running tryjobs.

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/01f121e242233f5fbbe2669d5d10df6fc0682949

commit 01f121e242233f5fbbe2669d5d10df6fc0682949
Author: Hans Wennborg <hans@chromium.org>
Date: Fri Sep 08 16:45:43 2017

Roll Clang 310694-2:312679

And remove the local patch.

Bug:  760615 , 755611 
Change-Id: Id131f345bdc7d30bbec00e22c562f2662e8bf7ed
Reviewed-on: https://chromium-review.googlesource.com/656062
Reviewed-by: Zachary Turner <zturner@chromium.org>
Commit-Queue: Hans Wennborg <hans@chromium.org>
Cr-Commit-Position: refs/heads/master@{#500608}
[delete] https://crrev.com/71a5f211c38e0f318896ea1492e5dbd82f176ec1/tools/clang/scripts/InstructionCombining.cpp
[modify] https://crrev.com/01f121e242233f5fbbe2669d5d10df6fc0682949/tools/clang/scripts/update.py

It seems to be sticking.