r295146 - Speculatively revert r295118 to see if it's what's causing the modules selfhost buildbots to fail.

2017-02-14 Thread Richard Smith via cfe-commits
Author: rsmith Date: Tue Feb 14 21:29:24 2017 New Revision: 295146 URL: http://llvm.org/viewvc/llvm-project?rev=295146&view=rev Log: Speculatively revert r295118 to see if it's what's causing the modules selfhost buildbots to fail. Modified: cfe/trunk/lib/AST/Type.cpp cfe/trunk/test/Sema

Re: r291955 - PR31606: Generalize our tentative DR resolution for inheriting copy/move

2017-02-14 Thread Richard Smith via cfe-commits
On 14 February 2017 at 18:14, Akira Hatanaka wrote: > Hi Richard, > > It looks like this commit causes an assertion failure when the following > code is compiled: > > $ cat test1.cpp > template > struct S3 { > }; > > template > struct S2 { > S2(S3 &&); > }; > > template > struct S1 : S2 { > u

r295149 - Fix assertion failure due to implicit special member lookup lacking a source location.

2017-02-14 Thread Richard Smith via cfe-commits
Author: rsmith Date: Tue Feb 14 22:18:23 2017 New Revision: 295149 URL: http://llvm.org/viewvc/llvm-project?rev=295149&view=rev Log: Fix assertion failure due to implicit special member lookup lacking a source location. Modified: cfe/trunk/lib/Sema/SemaLookup.cpp cfe/trunk/test/SemaCXX/c

Re: r291955 - PR31606: Generalize our tentative DR resolution for inheriting copy/move

2017-02-14 Thread Richard Smith via cfe-commits
On 14 February 2017 at 19:48, Richard Smith wrote: > On 14 February 2017 at 18:14, Akira Hatanaka wrote: > >> Hi Richard, >> >> It looks like this commit causes an assertion failure when the following >> code is compiled: >> >> $ cat test1.cpp >> template >> struct S3 { >> }; >> >> template >> s

Re: r291955 - PR31606: Generalize our tentative DR resolution for inheriting copy/move

2017-02-14 Thread Akira Hatanaka via cfe-commits
Thanks! > On Feb 14, 2017, at 8:31 PM, Richard Smith wrote: > > On 14 February 2017 at 19:48, Richard Smith > wrote: > On 14 February 2017 at 18:14, Akira Hatanaka > wrote: > Hi Richard, > > It looks like this commit causes an assertio

r295150 - [Sema] Disallow returning a __block variable via a move.

2017-02-14 Thread Akira Hatanaka via cfe-commits
Author: ahatanak Date: Tue Feb 14 23:15:28 2017 New Revision: 295150 URL: http://llvm.org/viewvc/llvm-project?rev=295150&view=rev Log: [Sema] Disallow returning a __block variable via a move. r274291 made changes to prefer calling a move constructor to calling a copy constructor when returning fr

[PATCH] D29908: Disallow returning a __block variable via a move

2017-02-14 Thread Akira Hatanaka via Phabricator via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rL295150: [Sema] Disallow returning a __block variable via a move. (authored by ahatanak). Changed prior to commit: https://reviews.llvm.org/D29908?vs=88430&id=88486#toc Repository: rL LLVM https://re

Re: r295114 - Improve diagnostic reporting when using __declspec without enabling __declspec as a keyword.

2017-02-14 Thread Mikael Holmén via cfe-commits
Hi Aaron, On 02/14/2017 11:47 PM, Aaron Ballman via cfe-commits wrote: Author: aaronballman Date: Tue Feb 14 16:47:20 2017 New Revision: 295114 URL: http://llvm.org/viewvc/llvm-project?rev=295114&view=rev Log: Improve diagnostic reporting when using __declspec without enabling __declspec as a

[libunwind] r295153 - [libunwind][CMake] Use libc++ headers when available

2017-02-14 Thread Petr Hosek via cfe-commits
Author: phosek Date: Wed Feb 15 00:41:57 2017 New Revision: 295153 URL: http://llvm.org/viewvc/llvm-project?rev=295153&view=rev Log: [libunwind][CMake] Use libc++ headers when available libunwind depends on C++ library headers. When building libunwind as part of LLVM and libc++ is available, use

<    1   2