[cfe-commits] r138526 - /cfe/trunk/lib/StaticAnalyzer/Core/CFRefCount.cpp
jediknil at belkadan.com
Thu Aug 25 11:55:30 PDT 2011
On Aug 25, 2011, at 1:52, Frits van Bommel wrote:
> On 25 August 2011 01:41, Jordy Rose <jediknil at belkadan.com> wrote:
>> + llvm_unreachable("Invalid/unknown GC mode.");
>> + return false;
> llvm_unreachable() expands to either __builtin_unreachable() or a call
> to a noreturn function. Either way, there should be no need for the
> explicit return.
Yeah, but the entire purpose of this is to silence a GCC warning about "missing return in a non-void function". Does GCC handle noreturn calls properly now? Anyway, I didn't want to take the chance. But thanks...if you think it's safe to remove the return than I can.
More information about the cfe-commits