r284366 - AST: Prefer LLVM_NODISCARD to LLVM_ATTRIBUTE_UNUSED_RESULT

2016-10-16 Thread Justin Bogner via cfe-commits
Author: bogner Date: Mon Oct 17 01:46:35 2016 New Revision: 284366 URL: http://llvm.org/viewvc/llvm-project?rev=284366&view=rev Log: AST: Prefer LLVM_NODISCARD to LLVM_ATTRIBUTE_UNUSED_RESULT Modified: cfe/trunk/lib/AST/ExprConstant.cpp Modified: cfe/trunk/lib/AST/ExprConstant.cpp URL: http

[PATCH] D24693: [CodeGen] Don't emit lifetime intrinsics for some local variables

2016-10-16 Thread Vitaly Buka via cfe-commits
vitalybuka marked an inline comment as done. vitalybuka added a comment. Please take a look. Meanwhile, I will investigate performance footprint. In https://reviews.llvm.org/D24693#559781, @rsmith wrote: > Is there some reasonable base set of functionality between this and > JumpDiagnostics tha

[PATCH] D24693: [CodeGen] Don't emit lifetime intrinsics for some local variables

2016-10-16 Thread Vitaly Buka via cfe-commits
vitalybuka updated this revision to Diff 74814. vitalybuka added a comment. - fixed comment - added test for indirect jumps https://reviews.llvm.org/D24693 Files: lib/CodeGen/CGDecl.cpp lib/CodeGen/CMakeLists.txt lib/CodeGen/CodeGenFunction.cpp lib/CodeGen/CodeGenFunction.h lib/CodeGe

[clang-tools-extra] r284360 - ClangMoveTests.cpp: Appease msc18.

2016-10-16 Thread NAKAMURA Takumi via cfe-commits
Author: chapuni Date: Mon Oct 17 00:09:58 2016 New Revision: 284360 URL: http://llvm.org/viewvc/llvm-project?rev=284360&view=rev Log: ClangMoveTests.cpp: Appease msc18. clang-tools-extra\unittests\clang-move\ClangMoveTests.cpp(216) : error C2593: 'operator =' is ambiguous llvm\includ

[clang-tools-extra] r284359 - Reformat.

2016-10-16 Thread NAKAMURA Takumi via cfe-commits
Author: chapuni Date: Mon Oct 17 00:08:49 2016 New Revision: 284359 URL: http://llvm.org/viewvc/llvm-project?rev=284359&view=rev Log: Reformat. Modified: clang-tools-extra/trunk/unittests/clang-move/ClangMoveTests.cpp Modified: clang-tools-extra/trunk/unittests/clang-move/ClangMoveTests.cpp

[PATCH] D24693: [CodeGen] Don't emit lifetime intrinsics for some local variables

2016-10-16 Thread Vitaly Buka via cfe-commits
vitalybuka updated this revision to Diff 74813. vitalybuka marked 5 inline comments as done. vitalybuka added a comment. Herald added a subscriber: modocache. - updated comments - indirect jumps - optimized Detect() https://reviews.llvm.org/D24693 Files: lib/CodeGen/CGDecl.cpp lib/CodeGen/C

r284355 - [CUDA] Fix false-positive in known-emitted handling.

2016-10-16 Thread Justin Lebar via cfe-commits
Author: jlebar Date: Sun Oct 16 21:25:55 2016 New Revision: 284355 URL: http://llvm.org/viewvc/llvm-project?rev=284355&view=rev Log: [CUDA] Fix false-positive in known-emitted handling. Previously: When compiling for host, our constructed call graph went *through* kernel calls. This meant that i

Re: r284137 - [ThinLTO] Update doc to include lld (now supported).

2016-10-16 Thread Davide Italiano via cfe-commits
On Sun, Oct 16, 2016 at 6:43 PM, Sean Silva wrote: > Nice to see this land! > > One nit: > Currently, doesn't LLD/ELF ignore -plugin-opt? That will mean that if a user > uses the "gold syntax" then LLD will silently ignore it, which isn't good. > At the very least, can we issue an error if we see

Re: r284137 - [ThinLTO] Update doc to include lld (now supported).

2016-10-16 Thread Sean Silva via cfe-commits
Nice to see this land! One nit: Currently, doesn't LLD/ELF ignore -plugin-opt? That will mean that if a user uses the "gold syntax" then LLD will silently ignore it, which isn't good. At the very least, can we issue an error if we see `-plugin-opt jobs=N` and suggest the LLD spelling? Or maybe ju

Re: [libcxx] r284214 - XFAIL aligned allocation tests for older Clang versions

2016-10-16 Thread Nico Weber via cfe-commits
See the Mac bot here: https://codereview.chromium.org/2416293003/ -> https://build.chromium.org/p/tryserver.chromium.mac/builders/mac_upload_clang/builds/105/steps/package%20clang/logs/stdio I think that bot uses Xcode 8's compiler. I haven't tried again since 2 days ago. On Sat, Oct 15, 2016 at 4

r284352 - clang/test/CXX/conv/conv.fctptr/p1.cpp: Appease for targeting i686-win32.

2016-10-16 Thread NAKAMURA Takumi via cfe-commits
Author: chapuni Date: Sun Oct 16 18:00:18 2016 New Revision: 284352 URL: http://llvm.org/viewvc/llvm-project?rev=284352&view=rev Log: clang/test/CXX/conv/conv.fctptr/p1.cpp: Appease for targeting i686-win32. error: 'error' diagnostics seen but not expected: File clang\test\CXX\conv\conv.fct

r284351 - Revert "Revert "[analyzer] Make MallocChecker more robust against custom redeclarations""

2016-10-16 Thread Devin Coughlin via cfe-commits
Author: dcoughlin Date: Sun Oct 16 17:19:03 2016 New Revision: 284351 URL: http://llvm.org/viewvc/llvm-project?rev=284351&view=rev Log: Revert "Revert "[analyzer] Make MallocChecker more robust against custom redeclarations"" This reverts commit r284340 to reapply r284335. The bot breakage was d

r284341 - AST: Improve a couple of comments and cast unused values to void

2016-10-16 Thread Justin Bogner via cfe-commits
Author: bogner Date: Sun Oct 16 15:12:42 2016 New Revision: 284341 URL: http://llvm.org/viewvc/llvm-project?rev=284341&view=rev Log: AST: Improve a couple of comments and cast unused values to void Make these comments a bit more explicit that they're initializing the RawText member, and explicitl

Re: r284335 - [analyzer] Make MallocChecker more robust against custom redeclarations

2016-10-16 Thread Devin Coughlin via cfe-commits
> On Oct 16, 2016, at 12:53 PM, Renato Golin wrote: > > On 16 October 2016 at 20:36, Devin Coughlin wrote: >> Reverted in r284340. > > Hi Devin, > > Sometimes Clang patches break on stage 2 or test-suite, but this is > not the case. I dug deeper and found that there was another commit, > not

Re: r284335 - [analyzer] Make MallocChecker more robust against custom redeclarations

2016-10-16 Thread Renato Golin via cfe-commits
On 16 October 2016 at 20:36, Devin Coughlin wrote: > Reverted in r284340. Hi Devin, Sometimes Clang patches break on stage 2 or test-suite, but this is not the case. I dug deeper and found that there was another commit, not showing on that range (it should have, but that's nothing to do with thi

Re: r284335 - [analyzer] Make MallocChecker more robust against custom redeclarations

2016-10-16 Thread Devin Coughlin via cfe-commits
> On Oct 16, 2016, at 12:12 PM, Devin Coughlin via cfe-commits > wrote: > >> >> On Oct 16, 2016, at 12:04 PM, Renato Golin wrote: >> >> On 16 October 2016 at 18:26, Devin Coughlin via cfe-commits >> wrote: >>> Author: dcoughlin >>> Date: Sun Oct 16 12:26:06 2016 >>> New Revision: 284335 >>>

r284340 - Revert "[analyzer] Make MallocChecker more robust against custom redeclarations"

2016-10-16 Thread Devin Coughlin via cfe-commits
Author: dcoughlin Date: Sun Oct 16 14:26:07 2016 New Revision: 284340 URL: http://llvm.org/viewvc/llvm-project?rev=284340&view=rev Log: Revert "[analyzer] Make MallocChecker more robust against custom redeclarations" This reverts commit r284335. It appears to be causing test-suite compile-time a

Re: r284335 - [analyzer] Make MallocChecker more robust against custom redeclarations

2016-10-16 Thread Devin Coughlin via cfe-commits
> On Oct 16, 2016, at 12:04 PM, Renato Golin wrote: > > On 16 October 2016 at 18:26, Devin Coughlin via cfe-commits > wrote: >> Author: dcoughlin >> Date: Sun Oct 16 12:26:06 2016 >> New Revision: 284335 >> >> URL: http://llvm.org/viewvc/llvm-project?rev=284335&view=rev >> Log: >> [analyzer] M

Re: r284335 - [analyzer] Make MallocChecker more robust against custom redeclarations

2016-10-16 Thread Renato Golin via cfe-commits
On 16 October 2016 at 18:26, Devin Coughlin via cfe-commits wrote: > Author: dcoughlin > Date: Sun Oct 16 12:26:06 2016 > New Revision: 284335 > > URL: http://llvm.org/viewvc/llvm-project?rev=284335&view=rev > Log: > [analyzer] Make MallocChecker more robust against custom redeclarations > > Add a

r284337 - P0012R1: Make exception specifications be part of the type system. This

2016-10-16 Thread Richard Smith via cfe-commits
Author: rsmith Date: Sun Oct 16 12:54:23 2016 New Revision: 284337 URL: http://llvm.org/viewvc/llvm-project?rev=284337&view=rev Log: P0012R1: Make exception specifications be part of the type system. This implements the bulk of the change (modifying the type system to include exception specificati

r284335 - [analyzer] Make MallocChecker more robust against custom redeclarations

2016-10-16 Thread Devin Coughlin via cfe-commits
Author: dcoughlin Date: Sun Oct 16 12:26:06 2016 New Revision: 284335 URL: http://llvm.org/viewvc/llvm-project?rev=284335&view=rev Log: [analyzer] Make MallocChecker more robust against custom redeclarations Add additional checking to MallocChecker to avoid crashing when memory routines have unex

[PATCH] D25654: [Sema] Don't perform aggregate initialization for types with explicit constructors

2016-10-16 Thread Richard Smith via cfe-commits
rsmith added a comment. Please also add a test to test/CXX/drs/dr15xx.cpp for core issue 1518, which this paper was resolving. Comment at: lib/AST/DeclCXX.cpp:564 +// C++1z [dcl.init.aggr]p1: +// - no user-provided, explicit, or inherited constructors, +if (getAST

[PATCH] D25661: [Driver] Support obtaining active toolchain from gcc-config on Gentoo

2016-10-16 Thread Michał Górny via cfe-commits
mgorny created this revision. mgorny added reviewers: rafael, chandlerc, bruno, bkramer. mgorny added a subscriber: cfe-commits. Support using gcc-config to determine the correct GCC toolchain location on Gentoo. In order to do that, attempt to read gcc-config configuration form [[sysroot]]/etc/en

[PATCH] D25660: [Analyzer] Checker for iterators dereferenced beyond their range.

2016-10-16 Thread Balogh , Ádám via cfe-commits
baloghadamsoftware created this revision. baloghadamsoftware added a reviewer: dcoughlin. baloghadamsoftware added subscribers: cfe-commits, xazax.hun, o.gyorgy. Herald added subscribers: modocache, mgorny, beanz. This checker checks for iterators dereferenced when they are equal to the end() of

[PATCH] D25659: [clang-tidy] Avoid running aliased checks twice

2016-10-16 Thread Malcolm Parsons via cfe-commits
malcolm.parsons created this revision. malcolm.parsons added reviewers: aaron.ballman, klimek, alexfh. malcolm.parsons added a subscriber: cfe-commits. Herald added a subscriber: nemanjai. It is easy to configure clang-tidy with both aliased and original checks enabled. This causes the check to b

[libcxx] r284333 - Make any_cast(nullptr) compile

2016-10-16 Thread Eric Fiselier via cfe-commits
Author: ericwf Date: Sun Oct 16 06:56:38 2016 New Revision: 284333 URL: http://llvm.org/viewvc/llvm-project?rev=284333&view=rev Log: Make any_cast(nullptr) compile Modified: libcxx/trunk/include/any libcxx/trunk/test/std/utilities/any/any.nonmembers/any.cast/any_cast_pointer.pass.cpp Mo

[PATCH] D25654: [Sema] Don't perform aggregate initialization for types with explicit constructors

2016-10-16 Thread Eric Fiselier via cfe-commits
EricWF created this revision. EricWF added a reviewer: rsmith. EricWF added a subscriber: cfe-commits. The C++17 rules for aggregate initialization changed to disallow types with explicit constructors [dcl.init.aggr]p1. This patch implements that new rule. https://reviews.llvm.org/D25654 Files

[PATCH] D25642: [clang-tidy] Use ignoreImplicit in cert-err58-cpp check

2016-10-16 Thread Malcolm Parsons via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rL284332: [clang-tidy] Use ignoreImplicit in cert-err58-cpp check (authored by malcolm.parsons). Changed prior to commit: https://reviews.llvm.org/D25642?vs=74770&id=74787#toc Repository: rL LLVM http

[clang-tools-extra] r284332 - [clang-tidy] Use ignoreImplicit in cert-err58-cpp check

2016-10-16 Thread Malcolm Parsons via cfe-commits
Author: malcolm.parsons Date: Sun Oct 16 04:47:10 2016 New Revision: 284332 URL: http://llvm.org/viewvc/llvm-project?rev=284332&view=rev Log: [clang-tidy] Use ignoreImplicit in cert-err58-cpp check Summary: Fix a false negative in cert-err58-cpp check when calling a constructor creates objects th