From cfe-commits Thu Jul 30 01:13:12 2015 From: Richard Smith Date: Thu, 30 Jul 2015 01:13:12 +0000 To: cfe-commits Subject: Re: Patches for 3.7 and move warnings Message-Id: X-MARC-Message: https://marc.info/?l=cfe-commits&m=143821895732701 MIME-Version: 1 Content-Type: multipart/mixed; boundary="--===============0204201228757098314==" --===============0204201228757098314== Content-Type: multipart/alternative; boundary=bcaec548a05f3722dc051c0d6853 --bcaec548a05f3722dc051c0d6853 Content-Type: text/plain; charset=UTF-8 Thanks, approved. On Wed, Jul 29, 2015 at 4:48 PM, Richard Trieu wrote: > The follow-up change is r243594. > > > On Wed, Jul 29, 2015 at 4:45 PM, Richard Smith > wrote: > >> On Wed, Jul 29, 2015 at 10:20 AM, Hans Wennborg >> 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 >>> 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 >>> >> >> > --bcaec548a05f3722dc051c0d6853 Content-Type: text/html; charset=UTF-8 Content-Transfer-Encoding: Quoted-printable
Thanks, approved.

On Wed, Jul 29, 2015 at 4:48 PM, Richard Trieu <rtrie= u@google.com> wrote:
The follow-up change is r243594.


On Wed, Jul 29, 2015 a= t 4:45 PM, Richard Smith <richard@metafoo.co.uk> wrote:<= br>
On Wed, Jul 29, 2015 at 10:20 AM, Hans = Wennborg <hwennborg@google.com> wrote:
Yes.

Richard Smith: approved for the branch?

<= div>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 warni= ngs.
> 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=3Drevision&revision=3D242600
> http://llvm.org/viewvc/llvm-project?view=3Drevision&revision=3D243463=
> http://llvm= .org/viewvc/llvm-project?view=3Drevision&revision=3D243538



--bcaec548a05f3722dc051c0d6853-- --===============0204201228757098314== Content-Type: text/plain; charset="us-ascii" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit Content-Disposition: inline _______________________________________________ cfe-commits mailing list cfe-commits@cs.uiuc.edu http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits --===============0204201228757098314==--