r180982 - Attempt to un-break the gdb buildbot.

Eric Christopher echristo at gmail.com
Tue May 7 15:23:45 PDT 2013


Actually you re-committed those parts, so still applies :)

-eric

On Tue, May 7, 2013 at 3:22 PM, Eric Christopher <echristo at gmail.com> wrote:
> Nevermind, I see you reverted it.
>
> -eric
>
> On Tue, May 7, 2013 at 3:19 PM, Eric Christopher <echristo at gmail.com> wrote:
>>> -    if (++NumStopPoints == 1)
>>> -      FirstStopPoint = Loc;
>>> +    //if (++NumStopPoints == 1)
>>> +      LastStopPoint = Loc;
>>>
>>
>> Eh? Commented out code? Also, side effects in an if conditional? Ew.
>>
>>> +  NumReturnExprs += 1;
>>>    if (RV == 0 || RV->isEvaluatable(getContext()))
>>> -    ++NumSimpleReturnExprs;
>>> +    NumSimpleReturnExprs += 1;
>>>
>>
>> Eh?
>>
>> -eric



More information about the cfe-commits mailing list