[libunwind] r308465 - Bump docs version to 6.0

Author: hans Date: Wed Jul 19 06:49:39 2017 New Revision: 308465 URL: http://llvm.org/viewvc/llvm-project?rev=308465=rev Log: Bump docs version to 6.0 Modified: libunwind/trunk/docs/conf.py Modified: libunwind/trunk/docs/conf.py URL:

[libcxx] r308462 - Bump docs version to 6.0

Author: hans Date: Wed Jul 19 06:46:11 2017 New Revision: 308462 URL: http://llvm.org/viewvc/llvm-project?rev=308462=rev Log: Bump docs version to 6.0 Modified: libcxx/trunk/docs/conf.py Modified: libcxx/trunk/docs/conf.py URL:

[libcxxabi] r308461 - Update version to 6.0.0svn

Author: hans Date: Wed Jul 19 06:42:18 2017 New Revision: 308461 URL: http://llvm.org/viewvc/llvm-project?rev=308461=rev Log: Update version to 6.0.0svn Modified: libcxxabi/trunk/CMakeLists.txt Modified: libcxxabi/trunk/CMakeLists.txt URL:

Re: r308441 - Recommit r308327: Add a warning for missing '#pragma pack (pop)'

On Wed, Jul 19, 2017 at 2:26 PM, Hans Wennborg wrote: > On Wed, Jul 19, 2017 at 1:30 PM, Alex Lorenz via cfe-commits > wrote: >> Author: arphaman >> Date: Wed Jul 19 04:30:41 2017 >> New Revision: 308441 >> >> URL:

r308455 - Revert r308441 "Recommit r308327: Add a warning for missing '#pragma pack (pop)' and suspicious uses of '#pragma pack' in included files"

Author: hans Date: Wed Jul 19 05:31:01 2017 New Revision: 308455 URL: http://llvm.org/viewvc/llvm-project?rev=308455=rev Log: Revert r308441 "Recommit r308327: Add a warning for missing '#pragma pack (pop)' and suspicious uses of '#pragma pack' in included files" This seems to have broken the

Re: r308441 - Recommit r308327: Add a warning for missing '#pragma pack (pop)'

On Wed, Jul 19, 2017 at 1:30 PM, Alex Lorenz via cfe-commits wrote: > Author: arphaman > Date: Wed Jul 19 04:30:41 2017 > New Revision: 308441 > > URL: http://llvm.org/viewvc/llvm-project?rev=308441=rev > Log: > Recommit r308327: Add a warning for missing '#pragma pack

[libunwind] r308454 - Creating release_50 branch off revision 308441

Author: hans Date: Wed Jul 19 05:20:59 2017 New Revision: 308454 URL: http://llvm.org/viewvc/llvm-project?rev=308454=rev Log: Creating release_50 branch off revision 308441 Added: libunwind/branches/release_50/ - copied from r308441, libunwind/trunk/

[libcxxabi] r308448 - Creating release_50 branch off revision 308441

Author: hans Date: Wed Jul 19 05:20:52 2017 New Revision: 308448 URL: http://llvm.org/viewvc/llvm-project?rev=308448=rev Log: Creating release_50 branch off revision 308441 Added: libcxxabi/branches/release_50/ - copied from r308441, libcxxabi/trunk/

[libcxx] r308447 - Creating release_50 branch off revision 308441

Author: hans Date: Wed Jul 19 05:20:52 2017 New Revision: 308447 URL: http://llvm.org/viewvc/llvm-project?rev=308447=rev Log: Creating release_50 branch off revision 308441 Added: libcxx/branches/release_50/ (props changed) - copied from r308441, libcxx/trunk/ Propchange:

r308427 - Update clang-cl options in the users manual

Author: hans Date: Wed Jul 19 02:52:24 2017 New Revision: 308427 URL: http://llvm.org/viewvc/llvm-project?rev=308427=rev Log: Update clang-cl options in the users manual Modified: cfe/trunk/docs/UsersManual.rst Modified: cfe/trunk/docs/UsersManual.rst URL:

Re: r303873 - Don't defer to the GCC driver for linking arm-baremetal

The test was failing on Windows; r303910 for that. On Thu, May 25, 2017 at 11:55 AM, Jonathan Roelofs via cfe-commits wrote: > > > On 5/25/17 11:39 AM, Galina Kistanova wrote: > > Hello Jonathan, > > This commit broke one of our builders: > >

r303910 - Make test/Driver/baremetal.cpp pass on Windows

Author: hans Date: Thu May 25 15:39:52 2017 New Revision: 303910 URL: http://llvm.org/viewvc/llvm-project?rev=303910=rev Log: Make test/Driver/baremetal.cpp pass on Windows Modified: cfe/trunk/test/Driver/baremetal.cpp Modified: cfe/trunk/test/Driver/baremetal.cpp URL:

r303267 - clang-cl: Fix path-based MSVC version detection

Author: hans Date: Wed May 17 10:27:44 2017 New Revision: 303267 URL: http://llvm.org/viewvc/llvm-project?rev=303267=rev Log: clang-cl: Fix path-based MSVC version detection The code wasn't taking the architecture-specific subdirectory into account. Differential Revision:

Re: r302547 - [mips] Impose a threshold for coercion of aggregates

On Tue, May 9, 2017 at 9:24 AM, Petar Jovanovic via cfe-commits wrote: > Author: petarj > Date: Tue May 9 11:24:03 2017 > New Revision: 302547 > > URL: http://llvm.org/viewvc/llvm-project?rev=302547=rev > Log: > [mips] Impose a threshold for coercion of aggregates > >

r302542 - Revert r302476 "Update testcase for upstream LLVM changes."

Author: hans Date: Tue May 9 10:55:39 2017 New Revision: 302542 URL: http://llvm.org/viewvc/llvm-project?rev=302542=rev Log: Revert r302476 "Update testcase for upstream LLVM changes." That test update was for r302469, which was reverted in r302533 due to PR32977. Modified:

r301567 - clang-cl: Alias /d1reportAllClassLayout to -fdump-record-layouts (PR32826)

Author: hans Date: Thu Apr 27 12:19:07 2017 New Revision: 301567 URL: http://llvm.org/viewvc/llvm-project?rev=301567=rev Log: clang-cl: Alias /d1reportAllClassLayout to -fdump-record-layouts (PR32826) Modified: cfe/trunk/include/clang/Driver/CLCompatOptions.td

r301564 - Fix -Wpedantic about extra semicolons in CGStmtOpenMP.cpp

Author: hans Date: Thu Apr 27 12:02:25 2017 New Revision: 301564 URL: http://llvm.org/viewvc/llvm-project?rev=301564=rev Log: Fix -Wpedantic about extra semicolons in CGStmtOpenMP.cpp Modified: cfe/trunk/lib/CodeGen/CGStmtOpenMP.cpp Modified: cfe/trunk/lib/CodeGen/CGStmtOpenMP.cpp URL:

Re: r301449 - PPCallbacks::MacroUndefined, change signature and add test.

The bots are red, e.g. http://lab.llvm.org:8011/builders/clang-x86-windows-msvc2015/builds/4342 On Wed, Apr 26, 2017 at 12:47 PM, Frederich Munch via cfe-commits wrote: > Author: marsupial > Date: Wed Apr 26 14:47:31 2017 > New Revision: 301449 > > URL:

r300501 - clang-cl: Support the /Zc:twoPhase[-] command-line option (PR32680)

Author: hans Date: Mon Apr 17 16:28:36 2017 New Revision: 300501 URL: http://llvm.org/viewvc/llvm-project?rev=300501=rev Log: clang-cl: Support the /Zc:twoPhase[-] command-line option (PR32680) It sounds like MSVC is adding support for two-phase name lookup in a future version, enabled by this

r300245 - clang-format-vs licence.txt: drop svn:executable

Author: hans Date: Thu Apr 13 15:09:18 2017 New Revision: 300245 URL: http://llvm.org/viewvc/llvm-project?rev=300245=rev Log: clang-format-vs licence.txt: drop svn:executable Not sure how it ended up with that property in the first place. Modified:

r300231 - Follow-up to r300225: update ClangFormat.csproj too

Author: hans Date: Thu Apr 13 13:15:00 2017 New Revision: 300231 URL: http://llvm.org/viewvc/llvm-project?rev=300231=rev Log: Follow-up to r300225: update ClangFormat.csproj too Modified: cfe/trunk/tools/clang-format-vs/ClangFormat/ClangFormat.csproj Modified:

r300226 - Follow-up to r300225: update ClangFormat.sln to VS2017

Author: hans Date: Thu Apr 13 12:47:47 2017 New Revision: 300226 URL: http://llvm.org/viewvc/llvm-project?rev=300226=rev Log: Follow-up to r300225: update ClangFormat.sln to VS2017 This got lost in the previous patch somehow. Modified: cfe/trunk/tools/clang-format-vs/ClangFormat.sln

r300225 - Warning-free clang-format plugin install for VS 15.0

Author: hans Date: Thu Apr 13 12:42:45 2017 New Revision: 300225 URL: http://llvm.org/viewvc/llvm-project?rev=300225=rev Log: Warning-free clang-format plugin install for VS 15.0 With the new release of VS, it's required that all plugins migrate to the new VSIX manifest format. The new format is

r300223 - clang-format-vs: Use a separate license.txt copy

Author: hans Date: Thu Apr 13 12:37:17 2017 New Revision: 300223 URL: http://llvm.org/viewvc/llvm-project?rev=300223=rev Log: clang-format-vs: Use a separate license.txt copy The regular file used to display very poorly in the VSIX installer due to long lines, wrapping etc. Added:

r300074 - Revert r300001 "Revert r298824 & r298816, recommit r298742 & r298754"

Author: hans Date: Wed Apr 12 11:40:26 2017 New Revision: 300074 URL: http://llvm.org/viewvc/llvm-project?rev=300074=rev Log: Revert r31 "Revert r298824 & r298816, recommit r298742 & r298754" It caused PR32640. Modified: cfe/trunk/include/clang/Basic/DiagnosticSerializationKinds.td

r299785 - Attempt to fix ms-intrinsics.c test

Author: hans Date: Fri Apr 7 12:01:56 2017 New Revision: 299785 URL: http://llvm.org/viewvc/llvm-project?rev=299785=rev Log: Attempt to fix ms-intrinsics.c test Modified: cfe/trunk/test/CodeGen/ms-intrinsics.c Modified: cfe/trunk/test/CodeGen/ms-intrinsics.c URL:

r299782 - Implement _interlockedbittestandset as a builtin

Author: hans Date: Fri Apr 7 11:41:47 2017 New Revision: 299782 URL: http://llvm.org/viewvc/llvm-project?rev=299782=rev Log: Implement _interlockedbittestandset as a builtin It's used by MS headers in VS 2017 without including intrin.h, so we can't implement it in the header anymore.

r299210 - clang-format vs plugin support for Visual Studio 2017

Author: hans Date: Fri Mar 31 07:50:42 2017 New Revision: 299210 URL: http://llvm.org/viewvc/llvm-project?rev=299210=rev Log: clang-format vs plugin support for Visual Studio 2017 With the release of Visual Studio 2017, we need to at the very least claim support for it in the current manifest

r298538 - [X86] Implement __readgsqword (and the rest) as builtins (PR32373)

Author: hans Date: Wed Mar 22 14:13:13 2017 New Revision: 298538 URL: http://llvm.org/viewvc/llvm-project?rev=298538=rev Log: [X86] Implement __readgsqword (and the rest) as builtins (PR32373) It seems MS headers have started using __readgsqword, and since it's used in a header that doesn't

Re: LLVM Lab SVN mirror is behind

r297634 was the 4.0.0 release commit. We'll probably see similar-size commits for 4.0.1 and 5.0.0. Can we make the buildbot svn mirror ignore that component of the repository to avoid this happening again? - Hans On Tue, Mar 14, 2017 at 12:23 AM, Galina Kistanova via cfe-commits

[libunwind] r297347 - Creating release candidate final from release_400 branch

Author: hans Date: Wed Mar 8 18:40:17 2017 New Revision: 297347 URL: http://llvm.org/viewvc/llvm-project?rev=297347=rev Log: Creating release candidate final from release_400 branch Added: libunwind/tags/RELEASE_400/final/ (props changed) - copied from r297346,

[libcxx] r297340 - Creating release candidate final from release_400 branch

Author: hans Date: Wed Mar 8 18:39:56 2017 New Revision: 297340 URL: http://llvm.org/viewvc/llvm-project?rev=297340=rev Log: Creating release candidate final from release_400 branch Added: libcxx/tags/RELEASE_400/final/ (props changed) - copied from r297339,

[libcxxabi] r297341 - Creating release candidate final from release_400 branch

Author: hans Date: Wed Mar 8 18:39:59 2017 New Revision: 297341 URL: http://llvm.org/viewvc/llvm-project?rev=297341=rev Log: Creating release candidate final from release_400 branch Added: libcxxabi/tags/RELEASE_400/final/ (props changed) - copied from r297340,

[libcxxabi] r297215 - Creating release candidate rc4 from release_400 branch

Author: hans Date: Tue Mar 7 14:58:42 2017 New Revision: 297215 URL: http://llvm.org/viewvc/llvm-project?rev=297215=rev Log: Creating release candidate rc4 from release_400 branch Added: libcxxabi/tags/RELEASE_400/rc4/ (props changed) - copied from r297214,

[libcxx] r297214 - Creating release candidate rc4 from release_400 branch

Author: hans Date: Tue Mar 7 14:58:39 2017 New Revision: 297214 URL: http://llvm.org/viewvc/llvm-project?rev=297214=rev Log: Creating release candidate rc4 from release_400 branch Added: libcxx/tags/RELEASE_400/rc4/ (props changed) - copied from r297213, libcxx/branches/release_40/

[libunwind] r297221 - Creating release candidate rc4 from release_400 branch

Author: hans Date: Tue Mar 7 14:59:01 2017 New Revision: 297221 URL: http://llvm.org/viewvc/llvm-project?rev=297221=rev Log: Creating release candidate rc4 from release_400 branch Added: libunwind/tags/RELEASE_400/rc4/ (props changed) - copied from r297220,

[libunwind] r296800 - Creating release candidate rc3 from release_400 branch

Author: hans Date: Thu Mar 2 13:31:35 2017 New Revision: 296800 URL: http://llvm.org/viewvc/llvm-project?rev=296800=rev Log: Creating release candidate rc3 from release_400 branch Added: libunwind/tags/RELEASE_400/rc3/ (props changed) - copied from r296799,

[libcxxabi] r296794 - Creating release candidate rc3 from release_400 branch

Author: hans Date: Thu Mar 2 13:31:16 2017 New Revision: 296794 URL: http://llvm.org/viewvc/llvm-project?rev=296794=rev Log: Creating release candidate rc3 from release_400 branch Added: libcxxabi/tags/RELEASE_400/rc3/ (props changed) - copied from r296793,

[libcxx] r296793 - Creating release candidate rc3 from release_400 branch

Author: hans Date: Thu Mar 2 13:31:13 2017 New Revision: 296793 URL: http://llvm.org/viewvc/llvm-project?rev=296793=rev Log: Creating release candidate rc3 from release_400 branch Added: libcxx/tags/RELEASE_400/rc3/ (props changed) - copied from r296792, libcxx/branches/release_40/

Re: r296656 - [PCH] Avoid VarDecl emission attempt if no owning module avaiable

Merged to 4.0 in r296762. On Wed, Mar 1, 2017 at 11:18 AM, Bruno Cardoso Lopes via cfe-commits wrote: > Author: bruno > Date: Wed Mar 1 13:18:42 2017 > New Revision: 296656 > > URL: http://llvm.org/viewvc/llvm-project?rev=296656=rev > Log: > [PCH] Avoid VarDecl

Re: [libcxx] r296561 - Fix PR32097 - is_abstract doesn't work on class templates.

We're at the "should have tagged 'final' days ago" stage :-) Since it's not a regression, I would prefer not to merge it unless you feel super strongly about it. Sounds like a good candidate for 4.0.1 (tracking bug is PR32061). Thanks, Hans On Wed, Mar 1, 2017 at 1:36 AM, Eric Fiselier

Re: [libunwind] r296358 - Add libcxxabi's LICENSE.TXT to libunwind.

Merged together with r296359 to 4.0 in r296364. On Mon, Feb 27, 2017 at 9:33 AM, Arnaud A. de Grandmaison via cfe-commits wrote: > Author: aadg > Date: Mon Feb 27 11:33:20 2017 > New Revision: 296358 > > URL: http://llvm.org/viewvc/llvm-project?rev=296358=rev > Log: >

[libunwind] r296364 - Merging r296358 and r296359:

Author: hans Date: Mon Feb 27 12:03:07 2017 New Revision: 296364 URL: http://llvm.org/viewvc/llvm-project?rev=296364=rev Log: Merging r296358 and r296359: r296358 | aadg | 2017-02-27 09:33:20 -0800 (Mon, 27 Feb 2017) | 8

Re: r293604 - In VirtualCallChecker, handle indirect calls

On Mon, Jan 30, 2017 at 9:23 PM, Sam McCall via cfe-commits wrote: > Author: sammccall > Date: Mon Jan 30 23:23:20 2017 > New Revision: 293604 > > URL: http://llvm.org/viewvc/llvm-project?rev=293604=rev > Log: > In VirtualCallChecker, handle indirect calls > > Summary:

Re: [PATCH] D29303: In VirtualCallChecker, handle indirect calls

Yes, this looks very straight-forward. Merged in r296154. On Fri, Feb 24, 2017 at 4:29 AM, Sam McCall via cfe-commits wrote: > Thanks Anna, I'm new to the release process here. > > Hans: this is a simple fix for a null-dereference in the static analyzer. > Does it

Re: r296063 - Revert r291477 "[Frontend] Correct values of ATOMIC_*_LOCK_FREE to match builtin"

Merged to 4.0 in r296152. On Thu, Feb 23, 2017 at 5:16 PM, Hans Wennborg via cfe-commits <cfe-commits@lists.llvm.org> wrote: > Author: hans > Date: Thu Feb 23 19:16:34 2017 > New Revision: 296063 > > URL: http://llvm.org/viewvc/llvm-project?rev=296063=rev > Log: >

Re: r295474 - [OpenMP] Fix cancellation point in task with no cancel

Merged to 4.0 in r296139 as requested on the r295473 commit thread. On Fri, Feb 17, 2017 at 10:32 AM, Jonas Hahnfeld via cfe-commits wrote: > Author: hahnfeld > Date: Fri Feb 17 12:32:58 2017 > New Revision: 295474 > > URL:

Re: r295473 - [OpenMP] Remove barriers at cancel and cancellation point

Oops, it did. Thanks for reminding me; I've merged that in r296139. On Thu, Feb 23, 2017 at 11:13 PM, Hahnfeld, Jonas wrote: > Hi Hans, > > Did r295474 fall off your radar? Sorry that I asked for both commits in one > email, should I reply to the other original

Re: r291477 - [Frontend] Correct values of ATOMIC_*_LOCK_FREE to match builtin

This was reverted in r296063 due to PR31864. On Mon, Jan 9, 2017 at 12:54 PM, Michal Gorny via cfe-commits wrote: > Author: mgorny > Date: Mon Jan 9 14:54:20 2017 > New Revision: 291477 > > URL: http://llvm.org/viewvc/llvm-project?rev=291477=rev > Log: > [Frontend]

r296063 - Revert r291477 "[Frontend] Correct values of ATOMIC_*_LOCK_FREE to match builtin"

Author: hans Date: Thu Feb 23 19:16:34 2017 New Revision: 296063 URL: http://llvm.org/viewvc/llvm-project?rev=296063=rev Log: Revert r291477 "[Frontend] Correct values of ATOMIC_*_LOCK_FREE to match builtin" It caused PR31864. There is a patch in progress to fix that, but let's revert in the

Re: r295843 - [OpenCL] r600 needs OpenCL kernel calling convention

Thanks! r296001. On Thu, Feb 23, 2017 at 2:01 AM, Anastasia Stulova wrote: > Sure! No objections here! > > Thanks, > Anastasia > > -Original Message- > From: hwennb...@google.com [mailto:hwennb...@google.com] On Behalf Of Hans > Wennborg > Sent: 22 February

Re: r295473 - [OpenMP] Remove barriers at cancel and cancellation point

Thanks! r296000. On Wed, Feb 22, 2017 at 8:15 PM, Alexey Bataev wrote: > Yes, approved > > Best regards, > Alexey Bataev > >> 23 февр. 2017 г., в 1:00, Hans Wennborg написал(а): >> >> Alexey: ping? >> >>> On Tue, Feb 21, 2017 at 11:07 AM, Hans Wennborg

Re: r295473 - [OpenMP] Remove barriers at cancel and cancellation point

Alexey: ping? On Tue, Feb 21, 2017 at 11:07 AM, Hans Wennborg wrote: > I'm Ok with it if Alexey approves. > > On Fri, Feb 17, 2017 at 10:52 AM, Hahnfeld, Jonas > wrote: >> Hi Hans, Alexey, >> >> can we merge this commit and r295474 for the 4.0

Re: r295843 - [OpenCL] r600 needs OpenCL kernel calling convention

I'm OK with it if either Matt or Anastasia agrees. Thanks, Hans On Wed, Feb 22, 2017 at 7:20 AM, Jan Vesely wrote: > Hi Hans, > > I'd like this commit to make it to 4.0. I'm not sure if it falls under > OpenCL or AMDGPU so I've added both Matt and Anastasia to cc. > >

Re: r295592 - clang/CMakeLists.txt: Rework r294954 -- use file(TO_CMAKE_PATH).

Merged in r295760. On Mon, Feb 20, 2017 at 3:12 PM, NAKAMURA Takumi via cfe-commits wrote: > Hans, could you pull this (and r294954) into release_40, please? > > This is a regression from previous releases that clang standalone build > cannot be configured on msbuild.

Re: r295610 - Link static PIE programs against rcrt0.o on OpenBSD

Merged to 4.0 in r295752 as requested in PR32013. On Sun, Feb 19, 2017 at 11:33 AM, Brad Smith via cfe-commits wrote: > Author: brad > Date: Sun Feb 19 13:33:26 2017 > New Revision: 295610 > > URL: http://llvm.org/viewvc/llvm-project?rev=295610=rev > Log: > Link

Re: r295473 - [OpenMP] Remove barriers at cancel and cancellation point

I'm Ok with it if Alexey approves. On Fri, Feb 17, 2017 at 10:52 AM, Hahnfeld, Jonas wrote: > Hi Hans, Alexey, > > can we merge this commit and r295474 for the 4.0 release or is it already > too late for that? I will totally understand that and can apply these >

Re: [PATCH] D29753: [PCH] Avoid early VarDecl emission attempt if no owning module avaiable

Richard, can you take a look when you have a moment? The PR is marked as a release blocker. On Thu, Feb 9, 2017 at 1:54 PM, Duncan P. N. Exon Smith via Phabricator via cfe-commits wrote: > dexonsmith added a comment. > > I'm not comfortable signing off on this, but it

Re: r295224 - PR24440: Do not silently discard a fold-expression appearing as the operand of a cast-expression.

Merged in r295375. Thanks, Hans On Wed, Feb 15, 2017 at 12:14 PM, Richard Smith wrote: > Hans, this would be a good candidate for Clang 4. The bug in question is not > a regression, but it is an accepts-invalid / wrong-code bug. > > On 15 February 2017 at 11:57, Richard

Re: D29829: [OpenCL][Doc] Description for adding OpenCL vendor extension in user manual

I've merged it in r295340. Cheers, Hans On Thu, Feb 16, 2017 at 6:26 AM, Anastasia Stulova wrote: > Hans, could we merge this documentation only change (r295313) in release40 > branch. I can commit myself if needed. :) > > Thanks in advance, > Anastasia > >

r295258 - [dllimport] Check for dtor references in functions

Author: hans Date: Wed Feb 15 17:28:10 2017 New Revision: 295258 URL: http://llvm.org/viewvc/llvm-project?rev=295258=rev Log: [dllimport] Check for dtor references in functions Destructor references are not modelled explicitly in the AST. This adds checks for destructor calls due to variable

r295257 - [dllimport] Look through typedefs and arrays in HasNonDllImportDtor

Author: hans Date: Wed Feb 15 17:28:07 2017 New Revision: 295257 URL: http://llvm.org/viewvc/llvm-project?rev=295257=rev Log: [dllimport] Look through typedefs and arrays in HasNonDllImportDtor The function is used to check whether a type is a class with non-dllimport destructor. It needs to

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

On Wed, Feb 15, 2017 at 12:17 PM, Richard Smith wrote: > On 15 February 2017 at 11:50, Hans Wennborg wrote: >> >> +Richard for risk/reward analysis. > > > This is an extremely safe change, and fixes what amounts to a subtle > miscompile. I think we

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

On Wed, Feb 15, 2017 at 12:14 PM, Richard Smith wrote: > On 15 February 2017 at 11:39, Hans Wennborg wrote: >> >> IIUC, this is a follow-up to r291955, which was merged to 4.0. Should >> this one be merged also? > > > Yes, please. r295233. > >> >>

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

+Richard for risk/reward analysis. r274291 was also in 3.9, so this isn't strictly speaking a regression. On Wed, Feb 15, 2017 at 11:43 AM, Akira Hatanaka wrote: > Hans, > > Can this be merged to 4.0 too? > >> On Feb 14, 2017, at 9:15 PM, Akira Hatanaka via cfe-commits >>

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

IIUC, this is a follow-up to r291955, which was merged to 4.0. Should this one be merged also? Cheers, Hans On Tue, Feb 14, 2017 at 8:18 PM, Richard Smith via cfe-commits wrote: > Author: rsmith > Date: Tue Feb 14 22:18:23 2017 > New Revision: 295149 > > URL:

Re: [libcxx] r294431 - [test] Fix hard_link_count test to account for fs with dir nlink==1

Michal requested this to be merged in PR31965, it seems fine to me, and Marshall OK'd it in an email to me. Merged in r295214. On Wed, Feb 8, 2017 at 1:57 AM, Michal Gorny via cfe-commits wrote: > Author: mgorny > Date: Wed Feb 8 03:57:32 2017 > New Revision: 294431

[libcxx] r295214 - Merging r294431:

Author: hans Date: Wed Feb 15 12:40:52 2017 New Revision: 295214 URL: http://llvm.org/viewvc/llvm-project?rev=295214=rev Log: Merging r294431: r294431 | mgorny | 2017-02-08 01:57:32 -0800 (Wed, 08 Feb 2017) | 7 lines [test]

Re: r294800 - Don't let EvaluationModes dictate whether an invalid base is OK

Merged in r295087. On Fri, Feb 10, 2017 at 5:00 PM, Hans Wennborg wrote: > Sgtm. Go ahead and merge when the bots have chewed on it for a bit, > otherwise I'll do it next week. > > Thanks, > Hans > > On Fri, Feb 10, 2017 at 3:06 PM, George Burgess IV >

Re: [libcxx] r294696 - Fully qualify (preprend ::) calls to math functions from libc

I'm guessing this isn't addressing a new issue in 4.0, so perhaps we should take the safe alternative and not merge this one. Thanks, Hans On Thu, Feb 9, 2017 at 11:35 PM, Eric Fiselier wrote: > I'm a bit paranoid one of the names we just qualified will end up being > defined as a

Re: r294855 - docs: update docs for objc_storeStrong behaviour

Sure, r295076. Thanks, Hans On Sat, Feb 11, 2017 at 12:29 PM, Saleem Abdulrasool wrote: > Hi Hans, > > Would you mind grabbing this for the 4.0 release as well? It's merely > correcting the documentation, so should have no impact on the toolchain > itself. > > On Sat,

Re: r294800 - Don't let EvaluationModes dictate whether an invalid base is OK

Sgtm. Go ahead and merge when the bots have chewed on it for a bit, otherwise I'll do it next week. Thanks, Hans On Fri, Feb 10, 2017 at 3:06 PM, George Burgess IV wrote: > Hi Hans! > > This fixes PR31843, which is a release blocker. Once the bots seem happy > with

Re: [libcxx] r294612 - Fix PR31916 - std::visit rejects visitors accepting lvalue arguments

Thanks! On Thu, Feb 9, 2017 at 4:49 PM, Eric Fiselier wrote: > Looks like it is just this change. I merged it in r294681. > > /Eric ___ cfe-commits mailing list cfe-commits@lists.llvm.org

r294658 - Don't try to link to the 4.0 release notes

Author: hans Date: Thu Feb 9 17:26:34 2017 New Revision: 294658 URL: http://llvm.org/viewvc/llvm-project?rev=294658=rev Log: Don't try to link to the 4.0 release notes Modified: cfe/trunk/docs/ReleaseNotes.rst Modified: cfe/trunk/docs/ReleaseNotes.rst URL:

[clang-tools-extra] r294657 - Don't try to link to the 4.0 release notes

Author: hans Date: Thu Feb 9 17:25:52 2017 New Revision: 294657 URL: http://llvm.org/viewvc/llvm-project?rev=294657=rev Log: Don't try to link to the 4.0 release notes Modified: clang-tools-extra/trunk/docs/ReleaseNotes.rst Modified: clang-tools-extra/trunk/docs/ReleaseNotes.rst URL:

Re: [libcxx] r294612 - Fix PR31916 - std::visit rejects visitors accepting lvalue arguments

Sounds good to me. You said "these changes"; are there more than this one? On Thu, Feb 9, 2017 at 11:15 AM, Eric Fiselier wrote: > I'm planning to merge this fix into the 4.0 release branch. > > Although not a regression, since is a new feature, it would still > be nice to ship

[libunwind] r294547 - Creating release candidate rc2 from release_400 branch

Author: hans Date: Wed Feb 8 18:14:34 2017 New Revision: 294547 URL: http://llvm.org/viewvc/llvm-project?rev=294547=rev Log: Creating release candidate rc2 from release_400 branch Added: libunwind/tags/RELEASE_400/rc2/ (props changed) - copied from r294546,

[libcxx] r294540 - Creating release candidate rc2 from release_400 branch

Author: hans Date: Wed Feb 8 18:14:12 2017 New Revision: 294540 URL: http://llvm.org/viewvc/llvm-project?rev=294540=rev Log: Creating release candidate rc2 from release_400 branch Added: libcxx/tags/RELEASE_400/rc2/ (props changed) - copied from r294539, libcxx/branches/release_40/

[libcxxabi] r294541 - Creating release candidate rc2 from release_400 branch

Author: hans Date: Wed Feb 8 18:14:16 2017 New Revision: 294541 URL: http://llvm.org/viewvc/llvm-project?rev=294541=rev Log: Creating release candidate rc2 from release_400 branch Added: libcxxabi/tags/RELEASE_400/rc2/ (props changed) - copied from r294540,

[libcxx] r294483 - Docs: drop minor version

Author: hans Date: Wed Feb 8 11:28:23 2017 New Revision: 294483 URL: http://llvm.org/viewvc/llvm-project?rev=294483=rev Log: Docs: drop minor version Modified: libcxx/branches/release_40/docs/conf.py Modified: libcxx/branches/release_40/docs/conf.py URL:

Re: [libcxx] r294138 - Adjust Linux ABI list after r294133

Ping? On Mon, Feb 6, 2017 at 2:15 PM, Hans Wennborg wrote: > The diff here is huge compared to the Apple one (r294139). > > Did the file format change and this file was previously not updated? > > This makes it hard to merge the change. Can you advise on how to > generate this

Re: [libcxx] r294138 - Adjust Linux ABI list after r294133

The diff here is huge compared to the Apple one (r294139). Did the file format change and this file was previously not updated? This makes it hard to merge the change. Can you advise on how to generate this file or how to do the same change on 4.0? On Sun, Feb 5, 2017 at 12:14 PM, Eric Fiselier

Re: [libcxx] r294133 - Change the base class of std::bad_optional_access. This is a (subtle) ABI change, and is in response to http://http://wg21.link/LWG2806, which I *expect* to be adopted in Kona.

I've merged this, together with your r294142, to 4.0 in r294249, as requested. If I understand correctly, we should also merge Eric's follow-ups r294138 and r294139, but I get merge conflicts for the Linux one so I've asked about how to handle that. Thanks, Hans On Sun, Feb 5, 2017 at 12:06 PM,

[libcxx] r294249 - Merging r294133 and r294142:

Author: hans Date: Mon Feb 6 15:59:19 2017 New Revision: 294249 URL: http://llvm.org/viewvc/llvm-project?rev=294249=rev Log: Merging r294133 and r294142: r294133 | marshall | 2017-02-05 12:06:38 -0800 (Sun, 05 Feb 2017) | 1

Re: r294008 - [Sema][ObjC++] Typo correction should handle ivars and properties

Thanks! r294059. On Fri, Feb 3, 2017 at 2:29 PM, Richard Smith wrote: > It looks like the only cases it should have any real impact on are those > where we would previously miscompile/crash, so this seems OK to me to merge > to Clang 4. > > > On 3 February 2017 at 13:45,

Re: r293043 - [analyzer] Fix MacOSXAPIChecker fp with static locals seen from nested blocks.

r294050. Thanks, Hans On Fri, Feb 3, 2017 at 1:50 PM, Anna Zaks wrote: > Fine with merging. Thank you! > Anna. >> On Feb 1, 2017, at 11:00 AM, Hans Wennborg wrote: >> >> If Anna is Ok with it, I'm fine with merging. >> >> Thanks, >> Hans >> >> On Wed, Feb 1,

Re: r294008 - [Sema][ObjC++] Typo correction should handle ivars and properties

IIUC, this isn't strictly fixing a regression from 3.9, but it looks like a pretty small diff. Richard, what do you think? On Fri, Feb 3, 2017 at 6:37 AM, Alex L wrote: > Hi Hans, > > Is there any chance we can merge this for 4.0? It fixed a nasty bug where > clang didn't

Re: r293043 - [analyzer] Fix MacOSXAPIChecker fp with static locals seen from nested blocks.

Ping? On Wed, Feb 1, 2017 at 11:00 AM, Hans Wennborg wrote: > If Anna is Ok with it, I'm fine with merging. > > Thanks, > Hans > > On Wed, Feb 1, 2017 at 10:29 AM, Artem Dergachev wrote: >> Hans, >> >> This is a fixed and tested version of the

Re: [libcxxabi] r292638 - Fix catch_reference_nullptr.pass.cpp test for GCC.

Merged this one in r293813. Cheers, Hans On Wed, Feb 1, 2017 at 1:16 PM, Renato Golin wrote: > Right, the other one was 292607 and has already been merged. Sorry for > the confusion. We just need to merge this one and we're done. > > --renato > > On 1 February 2017 at

[libcxxabi] r293813 - Merging r292638:

Author: hans Date: Wed Feb 1 15:30:36 2017 New Revision: 293813 URL: http://llvm.org/viewvc/llvm-project?rev=293813=rev Log: Merging r292638: r292638 | ericwf | 2017-01-20 11:34:19 -0800 (Fri, 20 Jan 2017) | 9 lines Fix

Re: [libcxxabi] r292638 - Fix catch_reference_nullptr.pass.cpp test for GCC.

On Tue, Jan 31, 2017 at 11:40 AM, Renato Golin wrote: > On 31 January 2017 at 18:36, Hans Wennborg wrote: Can you attach all commits that need to be backported into a bug against PR31622? We should probably squash them all before

r293800 - Drop 'dllimport' when redeclaring inline function template without the attribute (PR31695)

Author: hans Date: Wed Feb 1 12:52:53 2017 New Revision: 293800 URL: http://llvm.org/viewvc/llvm-project?rev=293800=rev Log: Drop 'dllimport' when redeclaring inline function template without the attribute (PR31695) For non-template dllimport functions, MSVC allows providing an inline

Re: r293043 - [analyzer] Fix MacOSXAPIChecker fp with static locals seen from nested blocks.

If Anna is Ok with it, I'm fine with merging. Thanks, Hans On Wed, Feb 1, 2017 at 10:29 AM, Artem Dergachev wrote: > Hans, > > This is a fixed and tested version of the previously-merged-and-reverted > r292800, do we still have time to land this into 4.0? > > Thanks, >

Re: r293787 - [CodeGen][ObjC] Avoid asserting on block pointer types in

OK. Merged in r293797. Thanks, Hans On Wed, Feb 1, 2017 at 9:50 AM, Alex L wrote: > Hi Hans, > > Would it be possible to merge this for 4.0? > > Cheers, > Alex > > On 1 February 2017 at 17:37, Alex Lorenz via cfe-commits > wrote: >> >> Author:

Re: r293360 - Change how we handle diagnose_if attributes.

Merged in r293784, as discussed on the commit thread for r291963. Thanks, Hans On Sun, Jan 29, 2017 at 5:59 PM, George Burgess IV wrote: > Hi! > > Now that the buidbots seem content, I think that this (...along with the > minor fix to the test in r293369) should be

Re: r291963 - [clang] Emit `diagnose_if` warnings from system headers

Merged this (r291963) in r293783. And the others (r293360 + r293369) in r293784. Thanks, Hans On Tue, Jan 31, 2017 at 7:17 PM, Richard Smith wrote: > I'm fine with these patches being merged. Hopefully we still have plenty of > time to shake out any problems between

Re: [PATCH] D24969: [Sema] Use the instantiated name of destructors in FindInstantiatedDecl and RebuildMemberExpr

On Tue, Jan 31, 2017 at 3:23 PM, Richard Smith wrote: > On 31 January 2017 at 14:49, Hans Wennborg wrote: >> >> Richard, what do you think? I don't believe this fixes a 3.9 >> regression, but the fix looks small, so it might be worth it. > > > I think

r293735 - Follow-up to r293732: add a proper triple to the test

Author: hans Date: Tue Jan 31 20:48:02 2017 New Revision: 293735 URL: http://llvm.org/viewvc/llvm-project?rev=293735=rev Log: Follow-up to r293732: add a proper triple to the test Modified: cfe/trunk/test/CodeGenCXX/initializer-list-ctor-order.cpp Modified:

r293732 - clang-cl: Evaluate arguments left-to-right in constructor call with initializer list (PR31831)

Author: hans Date: Tue Jan 31 20:21:07 2017 New Revision: 293732 URL: http://llvm.org/viewvc/llvm-project?rev=293732=rev Log: clang-cl: Evaluate arguments left-to-right in constructor call with initializer list (PR31831) clang-cl would evaluate the arguments right-to-left (see PR), and for

Re: r291963 - [clang] Emit `diagnose_if` warnings from system headers

I'm Ok with taking the larger patch (r293360 + r293369) too. It's been in tree for a bit, there is still a number of weeks before the release, and IIUC the major risk is that diagnose_if itself turns out to be broken, not that we'd miscompile anything? On Tue, Jan 31, 2017 at 11:11 AM, Richard

<    4   5   6   7   8   9   10   11   12   13   >