Checked in with fix as r159159.<br><br><div class="gmail_quote">On Fri, Jun 22, 2012 at 5:00 PM, Chandler Carruth <span dir="ltr"><<a href="mailto:chandlerc@google.com" target="_blank">chandlerc@google.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div style="font-family:arial,helvetica,sans-serif"><font><div class="gmail_quote"><div><div class="h5">On Fri, Jun 22, 2012 at 4:57 PM, Matt Beaumont-Gay <span dir="ltr"><<a href="mailto:matthewbg@google.com" target="_blank">matthewbg@google.com</a>></span> wrote:<br>

<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div><div>On Fri, Jun 22, 2012 at 4:51 PM, Chandler Carruth <<a href="mailto:chandlerc@google.com" target="_blank">chandlerc@google.com</a>> wrote:<br>


> On Fri, Jun 22, 2012 at 4:21 PM, Sam Panzer <<a href="mailto:panzer@google.com" target="_blank">panzer@google.com</a>> wrote:<br>
>><br>
>> This has been fixed in the attached patch, which will look nearly<br>
>> identical as the only change was to move a return statement out of one<br>
>> enclosing if-else block.<br>
><br>
><br>
> Did you add a test case to cover this case?<br>
<br>
</div></div>Rafael added a test case when he reverted the original.<br></blockquote><div><br></div></div></div><div>Lovely! =D</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">


<span><font color="#888888"><br>
-Matt<br>
</font></span></blockquote></div><br></font></div>
</blockquote></div><br>