[cfe-commits] r140232 - /cfe/trunk/lib/Sema/SemaExpr.cpp

[David Blaikie]

>
> I don't think that's the correct solution for many of these cases
>>
>
One other point on this - the issues you've raised already exist in both
assert and llvm_unreachable code, making this change (assert(0) ->
llvm_unreachable) somewhat orthogonal to them.

Though I'm happy to address them too, once we figure out which way to go,
either before or after this change. (& I'd also like to make similar changes
to llvm too, if we want to include them in the discussion and eventually
enshrine these decisions in the dev documentation)

- David
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20110921/e8dac40f/attachment.html>