r213415 - Merge two lines

Alexey Samsonov vonosmas at gmail.com
Fri Jul 18 18:05:21 PDT 2014


Thank you! (and sorry for breakage).


On Fri, Jul 18, 2014 at 4:57 PM, NAKAMURA Takumi <geek4civic at gmail.com>
wrote:

> Excuse me, reverted in r213431.
>
> CGBuilder doesn't name with -Asserts. setName() is still required here.
>
> 2014-07-19 6:29 GMT+09:00 Alexey Samsonov <vonosmas at gmail.com>:
> > Author: samsonov
> > Date: Fri Jul 18 16:29:55 2014
> > New Revision: 213415
> >
> > URL: http://llvm.org/viewvc/llvm-project?rev=213415&view=rev
> > Log:
> > Merge two lines
> >
> > Modified:
> >     cfe/trunk/lib/CodeGen/CGDecl.cpp
> >
> > Modified: cfe/trunk/lib/CodeGen/CGDecl.cpp
> > URL:
> http://llvm.org/viewvc/llvm-project/cfe/trunk/lib/CodeGen/CGDecl.cpp?rev=213415&r1=213414&r2=213415&view=diff
> >
> ==============================================================================
> > --- cfe/trunk/lib/CodeGen/CGDecl.cpp (original)
> > +++ cfe/trunk/lib/CodeGen/CGDecl.cpp Fri Jul 18 16:29:55 2014
> > @@ -907,8 +907,7 @@ CodeGenFunction::EmitAutoVarAlloca(const
> >        if (isByRef)
> >          LTy = BuildByRefType(&D);
> >
> > -      llvm::AllocaInst *Alloc = CreateTempAlloca(LTy);
> > -      Alloc->setName(D.getName());
> > +      llvm::AllocaInst *Alloc = CreateTempAlloca(LTy, D.getName());
> >
> >        CharUnits allocaAlignment = alignment;
> >        if (isByRef)
> >
> >
> > _______________________________________________
> > cfe-commits mailing list
> > cfe-commits at cs.uiuc.edu
> > http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits
>



-- 
Alexey Samsonov
vonosmas at gmail.com
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20140718/cfb0a618/attachment.html>


More information about the cfe-commits mailing list