Patches for 3.7 and move warnings

Hans Wennborg hwennborg at google.com
Wed Jul 29 18:57:38 PDT 2015


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 at metafoo.co.uk> wrote:
> Thanks, approved.
>
>
> On Wed, Jul 29, 2015 at 4:48 PM, Richard Trieu <rtrieu at google.com> wrote:
>>
>> The follow-up change is r243594.
>>
>>
>> On Wed, Jul 29, 2015 at 4:45 PM, Richard Smith <richard at metafoo.co.uk>
>> wrote:
>>>
>>> On Wed, Jul 29, 2015 at 10:20 AM, Hans Wennborg <hwennborg at 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 at 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
>>>
>>>
>>
>



More information about the cfe-commits mailing list