[prev in list] [next in list] [prev in thread] [next in thread] 

List:       cfe-commits
Subject:    Re: Patches for 3.7 and move warnings
From:       Hans Wennborg <hwennborg () google ! com>
Date:       2015-07-30 1:57:38
Message-ID: CAB8jPhdyXSEcaVVyaqVBLOxvAYio7but81uzhiB_P+oB9MtZxQ () mail ! gmail ! com
[Download RAW message or body]

Merged:

r242600 in r243605.
(Also grabbed r242854 to avoid merge conflicts; r243606.)
r243463, r243538, and r243594 in r243607.

Thanks,
Hans

On Wed, Jul 29, 2015 at 6:13 PM, Richard Smith <richard@metafoo.co.uk> wrote:
> Thanks, approved.
>
>
> On Wed, Jul 29, 2015 at 4:48 PM, Richard Trieu <rtrieu@google.com> wrote:
>>
>> The follow-up change is r243594.
>>
>>
>> On Wed, Jul 29, 2015 at 4:45 PM, Richard Smith <richard@metafoo.co.uk>
>> wrote:
>>>
>>> On Wed, Jul 29, 2015 at 10:20 AM, Hans Wennborg <hwennborg@google.com>
>>> wrote:
>>>>
>>>> Yes.
>>>>
>>>> Richard Smith: approved for the branch?
>>>
>>>
>>> First and third LGTM for branch. r243463 has a pending follow-up change,
>>> and we should take neither or both (preferably both).
>>>
>>>> On Wed, Jul 29, 2015 at 10:13 AM, Richard Trieu <rtrieu@google.com>
>>>> wrote:
>>>> > Hans,
>>>> >
>>>> > These are patches to fix -Wpessimizing-move and -Wredundant-move
>>>> > warnings.
>>>> > Is it still possible to get these into the 3.7 branch?
>>>> >
>>>> > Richard
>>>> >
>>>> > Revisions:
>>>> > r242600
>>>> > r243463
>>>> > r243538
>>>> >
>>>> > Links:
>>>> > http://llvm.org/viewvc/llvm-project?view=revision&revision=242600
>>>> > http://llvm.org/viewvc/llvm-project?view=revision&revision=243463
>>>> > http://llvm.org/viewvc/llvm-project?view=revision&revision=243538
>>>
>>>
>>
>
_______________________________________________
cfe-commits mailing list
cfe-commits@cs.uiuc.edu
http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits
[prev in list] [next in list] [prev in thread] [next in thread]