LLVM- release_35 backprt patch.

Daniel Sanders Daniel.Sanders at imgtec.com
Sat Dec 6 02:41:10 PST 2014


I know from another thread that Neeraj doesn't have commit access so I've merged them in r223581, and r223582. Thanks
________________________________________
From: Tom Stellard [tom at stellard.net]
Sent: 06 December 2014 02:32
To: Daniel Sanders
Cc: Neeraj Badlani; llvm-commits at cs.uiuc.edu; sabre at nondot.org
Subject: Re: LLVM- release_35 backprt patch.

These are both fine to merge to 3.5.  Do you have commit access?

On Fri, Dec 05, 2014 at 11:50:44AM +0000, Daniel Sanders wrote:
> Ok, so going by the git-svn-id's it's r220360, r215169. Both look good to me but I'm not the code owner for this area. We don't seem to have an explicit code owner for this area so it must be Chris Lattner.
> <https://llvm.org/svn/llvm-project/llvm/trunk@215169>
> ________________________________
> From: Neeraj Badlani [neerajbadlani at gmail.com]
> Sent: 03 December 2014 17:47
> To: Daniel Sanders
> Cc: llvm-commits at cs.uiuc.edu; Tom Stellard (tom at stellard.net)
> Subject: Re: LLVM- release_35 backprt patch.
>
> Hi ,
> Attached is got log from my local branch . I have highlighted two commits from which I cherry picked .
>
> ================================================================
> [llvm] git show 0cd299b
> commit 0cd299be0ca2af7a104c395e4300976512bd9757
> Author: Filipe Cabecinhas <me at filcab.net<mailto:me at filcab.net>>
> Date:   Wed Oct 22 02:16:06 2014 +0000
>
>     Silence gcc's -Wcomment
>
>     gcc's (4.7, I think) -Wcomment warning is not "as smart" as clang's and
>     warns even if the line right after the backslash-newline sequence only has
>     a line comment that starts at the beginning of the line.
>
>     git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@220360 91177308-0d34-0410-b5e6-96231b3b80d8
>     (cherry picked from commit 8ff1b9820829cfccd9866c54269efee93d68d8b6)
>     Signed-off-by: Neeraj Badlani <neerajbadlani at gmail.com<mailto:neerajbadlani at gmail.com>>
> ================================================================
>
> [llvm] git show edfc599
> commit edfc599a28a0a8190f31c942017ee81f629f3866
> Author: Gerolf Hoflehner <ghoflehner at apple.com<mailto:ghoflehner at apple.com>>
> Date:   Thu Aug 7 23:19:55 2014 +0000
>
>     Fix for multi-line comment warning
>
>     git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@215169 91177308-0d34-0410-b5e6-96231b3b80d8
>     (cherry picked from commit d0f88d538121f3e3d645aeed419803ee9a7d7da7)
>     Signed-off-by: Neeraj Badlani <neerajbadlani at gmail.com<mailto:neerajbadlani at gmail.com>>
>
>
> On Wed, Dec 3, 2014 at 9:39 AM, Daniel Sanders <Daniel.Sanders at imgtec.com<mailto:Daniel.Sanders at imgtec.com>> wrote:
> CC'd the release manager (Tom Stellard). It would be good to merge this in too. It fixes the warnings about multiline comments caused by the backslashes at the end of lines.
>
> We normally use a tool to merge commits directly from the trunk. Could you tell Tom the revision numbers you created your patch from?
>
> > -----Original Message-----
> > From: llvm-commits-bounces at cs.uiuc.edu<mailto:llvm-commits-bounces at cs.uiuc.edu> [mailto:llvm-commits-<mailto:llvm-commits->
> > bounces at cs.uiuc.edu<mailto:bounces at cs.uiuc.edu>] On Behalf Of Neeraj Badlani
> > Sent: 03 December 2014 15:33
> > To: llvm-commits at cs.uiuc.edu<mailto:llvm-commits at cs.uiuc.edu>
> > Subject: LLVM- release_35 backprt patch.
> >
> > Hi
> > I have just recently started learning LLVM , and during building of release_35
> > , I noticed couple of build errors .
> >
> > Those have been corrected in the master , but have not been backported
> > yet to release_35 branch.
> >
> > I cherry picked couple of commits from the master and applied to
> > release_35 .
> >
> > Patch is attached in mail .
> >
> > Since this is my first time submitting any patch in an open source
> > project , please let me know if I have done something wrong / can be
> > done better .
> >
> >
> > Thanks
>




More information about the llvm-commits mailing list