<div dir="ltr">Is a CompoundStatement's start/end loc ever different from lbrace/rbrace location? (maybe when it's a compound statement without braces? But that can never occur in this part of the AST, right) If it isn't, then the if/else LBrace/RBrace bit could be omitted. Maybe then we could just remove the "if CompoundStatement" bit entirely? (or check that it's not a defaulted definition, etc? Not sure what else might appear here in the AST)<br></div><div class="gmail_extra"><br><div class="gmail_quote">On Wed, Oct 22, 2014 at 1:37 PM, Aaron Ballman <span dir="ltr"><<a href="mailto:aaron@aaronballman.com" target="_blank">aaron@aaronballman.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Details, details! ;-) It was trivial, but not THAT trivial -- thanks, David!<br>
<span class="HOEnZb"><font color="#888888"><br>
~Aaron<br>
</font></span><div class="HOEnZb"><div class="h5"><br>
On Wed, Oct 22, 2014 at 4:34 PM, David Blaikie <<a href="mailto:dblaikie@gmail.com">dblaikie@gmail.com</a>> wrote:<br>
> (missing patch attachment)<br>
><br>
> On Wed, Oct 22, 2014 at 1:29 PM, Aaron Ballman <<a href="mailto:aaron@aaronballman.com">aaron@aaronballman.com</a>><br>
> wrote:<br>
>><br>
>> Currently, function try blocks are not analyzed to determine whether<br>
>> all control paths return a value. This patch addresses that, and adds<br>
>> test cases for it. This appears to be a relatively simple change<br>
>> required to support this.<br>
>><br>
>> ~Aaron<br>
>> _______________________________________________<br>
>> cfe-commits mailing list<br>
>> <a href="mailto:cfe-commits@cs.uiuc.edu">cfe-commits@cs.uiuc.edu</a><br>
>> <a href="http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits" target="_blank">http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits</a><br>
><br>
><br>
</div></div></blockquote></div><br></div>