[PATCH] Proposal on how to fix temporary dtors.

Manuel Klimek klimek at google.com
Sun Jun 15 14:53:34 PDT 2014


>>! In D3627#56, @jordan_rose wrote:
> At first I misread that as
> 
>   void testSequencingOfConditionalTempDtors(bool b) {
>      (b || check(Dtor())), check(NoReturnDtor());
>      clang_analyzer_warnIfReached();  // expected-warning{{REACHABLE}}
>    }
> 
> which should NOT be reachable. That's probably also a good test case. The comma operator is a real pain.

Funnily enough I had 10 minutes where I debugged why the CFG was looking incorrect, only to detect that in my small repro I had actually forgotten the parens for the comma operator.

http://reviews.llvm.org/D3627






More information about the cfe-commits mailing list