3.8 Merge Request: r257915

Tom Stellard via llvm-commits llvm-commits at lists.llvm.org
Wed May 18 13:39:43 PDT 2016


On Fri, May 13, 2016 at 05:33:03PM -0400, Rafael Espíndola wrote:
> approved.
> 

Hi,

This has been merged as r269978.

-Tom

> Cheers,
> Rafael
> 
> On 13 May 2016 at 16:53, Mehdi Amini <mehdi.amini at apple.com> wrote:
> > Some context: without this patch, we can't LTO-bootstrap clang with assertion enabled, because we end-up walking use-list of constant for lazy-loaded module: the assertions have been added in 3.8, but not the fix to make the build clean with the assertions.
> >
> > CC Rafael for approval.
> >
> > --
> > Mehdi
> >
> >
> >
> >> On May 13, 2016, at 1:50 PM, Chris Lattner <sabre at nondot.org> wrote:
> >>
> >> Yes, if Rafael approves, I’m ok with it.  He’s far more familiar with canBeOmittedFromSymbolTable than I am.
> >>
> >> -Chris
> >>
> >>> On May 13, 2016, at 12:22 PM, Tom Stellard <tom at stellard.net> wrote:
> >>>
> >>> Hi Chris,
> >>>
> >>> Are these patches OK to merge to the 3.8 branch?
> >>>
> >>> http://reviews.llvm.org/rL257915
> >>>
> >>> Thanks,
> >>> Tom
> >>
> >


More information about the llvm-commits mailing list