[llvm] r223500 - BFI: Saturate when combining edges to a successor

Duncan P. N. Exon Smith dexonsmith at apple.com
Mon Dec 8 15:30:25 PST 2014


> On 2014 Dec 8, at 13:15, Tom Stellard <tom at stellard.net> wrote:
> 
> On Mon, Dec 08, 2014 at 09:46:55PM +0100, Chandler Carruth wrote:
>> On Mon, Dec 8, 2014 at 6:51 PM, Duncan P. N. Exon Smith <
>> dexonsmith at apple.com> wrote:
>> 
>>> +atrick,chandlerc
>>> 
>>>> On 2014 Dec 8, at 09:29, Chris Lattner <sabre at nondot.org> wrote:
>>>> 
>>>> 
>>>>> On Dec 8, 2014, at 8:52 AM, Tom Stellard <tom at stellard.net> wrote:
>>>>> 
>>>>> On Fri, Dec 05, 2014 at 06:36:41PM -0800, Duncan P. N. Exon Smith wrote:
>>>>>> Tom,
>>>>>> 
>>>>>> Is this okay for 3.5?
>>>>>> 
>>>>> 
>>>>> cc'ing Chris who is the default code owner.
>>>> 
>>>> LGTM, though I’m not the best to review this.   Maybe we should get a
>>> profiling/coverage code owner?
>>> 
>>> I've CC'ed Andy and Chandler, who reviewed my work when I rewrote this
>>> analysis in the spring.
>>> 
>> 
>> This is clearly good for the release.
> 
> LGTM.
> 
> Go ahead and merge this to the 3.5 branch.
> 
> -Tom

r223716





More information about the llvm-commits mailing list