As per recent discussions on cfe-commits, here's a patch to change all sure-failing assertions to llvm_unreachables (this task was made easier by the canonicalization of these asserts that Richard's been doing with the new boolean conversion from string warning - which meant there were only two major cases ("assert(0 &&" and "assert(false &&" - I don't think I found any message-less asserts in clang though I think there are some in LLVM itself)). The only other thing I had to do was some 80 col wraps due to the increase in length.<div>
<br>If this is the right thing just say the word & I'll check it in.</div>