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

Tom Stellard tom at stellard.net
Mon Dec 8 13:15:41 PST 2014


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
> _______________________________________________
> 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