[PATCH] fix llvm linker issue with dbg.declare

Chad Rosier mcrosier at apple.com
Wed Jan 30 10:52:27 PST 2013


On Jan 30, 2013, at 10:37 AM, Eli Bendersky <eliben at google.com> wrote:

>>> Since I got no response on this patch and we need this patch soon, I committed at r173946.
>> 
>> Manman,
>> It may have been best to continue pressing for a review first.
>> 
> 
> Yes.
> 
>>>> If you have any concern about this patch, please review after commit.
>>>> 
>>> 
>>> Ah, it's good to know there is a new patch review policy for LLVM.
>>> Thanks, Manman.
>> 
>> Eli,
>> What was the point of this comment?
> 
> The point was to express the same thing you said above. The sarcastic
> tone may have been uncalled for, but stems from bitter experience.

I'm sorry you've had a number of poor experiences, but lets try to be constructive/positive despite them.

> Having had the pleasure (on multiple occasions  for multiple code
> areas) to chase code owners for weeks with multiple list & IRC pings
> before getting the LGTM, it hurts to see the honor code broken by a
> review-less commit made less than 48 hours after the initial request,
> because "we need this patch soon".

Thanks for the clarification, Eli.  I'm sure it's very frustrating.

> Eli
> _______________________________________________
> llvm-commits mailing list
> llvm-commits at cs.uiuc.edu
> http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits




More information about the llvm-commits mailing list