[PATCH] D37057: [clang] Require address space to be specified when creating functions (3/3)

2017-12-09 Thread Dylan McKay via Phabricator via cfe-commits
dylanmckay updated this revision to Diff 126270. dylanmckay added a comment. Rebase on trunk Repository: rC Clang https://reviews.llvm.org/D37057 Files: lib/CodeGen/CGBlocks.cpp lib/CodeGen/CGBuiltin.cpp lib/CodeGen/CGCUDANV.cpp lib/CodeGen/CGDeclCXX.cpp lib/CodeGen/CGException.cpp

r320250 - Switch to gnu++14 as the default dialect.

2017-12-09 Thread Tim Northover via cfe-commits
Author: tnorthover Date: Sat Dec 9 04:09:54 2017 New Revision: 320250 URL: http://llvm.org/viewvc/llvm-project?rev=320250&view=rev Log: Switch to gnu++14 as the default dialect. This is C++14 with conforming GNU extensions. Modified: cfe/trunk/docs/ReleaseNotes.rst cfe/trunk/lib/Fronten

[PATCH] D32411: [libcxx] Provide #include_next alternative for MSVC

2017-12-09 Thread Ben Craig via Phabricator via cfe-commits
bcraig updated this revision to Diff 126275. bcraig added a comment. Rebased https://reviews.llvm.org/D32411 Files: CMakeLists.txt docs/DesignDocs/IncludeNextEmulation.rst include/__config include/__config_site.in include/complex.h include/cstddef include/ctype.h include/errno.h

[PATCH] D41048: [libcxx] workaround PR 28385 in __find_exactly_one_checked

2017-12-09 Thread Casey Carter via Phabricator via cfe-commits
CaseyCarter created this revision. Fixes #35578. https://reviews.llvm.org/D41048 Files: include/tuple Index: include/tuple === --- include/tuple +++ include/tuple @@ -1012,10 +1012,21 @@ template struct __find_exactly_one_

[PATCH] D41048: [libcxx] workaround PR 28385 in __find_exactly_one_checked

2017-12-09 Thread Roman Lebedev via Phabricator via cfe-commits
lebedev.ri added a comment. Test? https://reviews.llvm.org/D41048 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[PATCH] D41048: [libcxx] workaround PR 28385 in __find_exactly_one_checked

2017-12-09 Thread Casey Carter via Phabricator via cfe-commits
CaseyCarter added a comment. In https://reviews.llvm.org/D41048#950424, @lebedev.ri wrote: > Test? Four of the existing variant tests fail without this change (Using the clang-6 dailies from apt.llvm.org and -std=c++2a): Failing Tests (4): libc++ :: std/utilities/variant/variant.get/get_

[PATCH] D30691: [analyzer] Support for naive cross translational unit analysis

2017-12-09 Thread George Karpenkov via Phabricator via cfe-commits
george.karpenkov added a comment. Python part looks good to me. I don't know whether @dcoughlin or @NoQ would want to insert additional comments on C++ parts. https://reviews.llvm.org/D30691 ___ cfe-commits mailing list cfe-commits@lists.llvm.org h

[PATCH] D41050: Fix over-release of return value of lambda implicitly converted to block

2017-12-09 Thread Dan Zimmerman via Phabricator via cfe-commits
danzimm created this revision. Herald added subscribers: cfe-commits, kosarev. Clang has a pretty cool feature right now that will allow you to use a lambda as a block. Unfortunately there's a bug in this conversion if the return value of the block is an ObjC object and arc is enabled -- the ret

[PATCH] D41050: Fix over-release of return value of lambda implicitly converted to block

2017-12-09 Thread Xin Tong via Phabricator via cfe-commits
trentxintong added a comment. @gottesmm can you please take a look ? Thanks. Repository: rC Clang https://reviews.llvm.org/D41050 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

r320271 - Update Clang CMake cache to use cxx-headers, NFC

2017-12-09 Thread Duncan P. N. Exon Smith via cfe-commits
Author: dexonsmith Date: Sat Dec 9 15:44:42 2017 New Revision: 320271 URL: http://llvm.org/viewvc/llvm-project?rev=320271&view=rev Log: Update Clang CMake cache to use cxx-headers, NFC Apparently libcxx-headers is going away. Fixes PR35584. Modified: cfe/trunk/cmake/caches/Apple-stage2.cma

Re: r320250 - Switch to gnu++14 as the default dialect.

2017-12-09 Thread Richard Smith via cfe-commits
Thank you and everyone else who has helped to make this happen! On 9 Dec 2017 04:10, "Tim Northover via cfe-commits" < cfe-commits@lists.llvm.org> wrote: > Author: tnorthover > Date: Sat Dec 9 04:09:54 2017 > New Revision: 320250 > > URL: http://llvm.org/viewvc/llvm-project?rev=320250&view=rev >

[PATCH] D41054: Teach clang/NetBSD about additional dependencies for sanitizers

2017-12-09 Thread Kamil Rytarowski via Phabricator via cfe-commits
krytarowski created this revision. krytarowski added a project: Sanitizers. Sanitizers on NetBSD require additional linkage: - libutil for forkpty(3) - libexecinfo for backtrace(3) Sponsored by Repository: rL LLVM https://reviews.llvm.org/D41054 Files: lib/Driver/ToolChains/CommonArgs.c

[PATCH] D41055: [debuginfo-tests] Add support for moving debuginfo-tests from clang/test to llvm/projects or monorepo.

2017-12-09 Thread Don Hinton via Phabricator via cfe-commits
hintonda created this revision. Herald added subscribers: cfe-commits, JDevlieghere, mgorny. The new version of debuginfo-tests will have it's own lit.cfg.py file which is incompatible with the one in clang/test. This change supports both the old and new versions, and can be used until the bots ac

[PATCH] D40998: [driver][darwin] Take the OS version specified in "-target" as the target OS instead of inferring it from SDK / environment

2017-12-09 Thread Bob Wilson via Phabricator via cfe-commits
bob.wilson requested changes to this revision. bob.wilson added inline comments. This revision now requires changes to proceed. Comment at: lib/Driver/ToolChains/Darwin.cpp:1518-1523 + // Warn about superfluous OS_DEPLOYMENT_TARGET environment variable. + Optional EnvTa