Re: [PATCH] D19118: [clang-tidy] Add checker for operations between integrals and pointers

2016-04-14 Thread Eugene Zelenko via cfe-commits
Eugene.Zelenko added a subscriber: Eugene.Zelenko. Comment at: docs/clang-tidy/checks/misc-pointer-and-integral-operation.rst:9 @@ +8,3 @@ +object is compare to an object with integral type. + + Looks like excessive line. Comment at: docs/clang-t

Re: [PATCH] D19006: [libcxx] Remove the "reduced-arity-initialization" extension from the uses-allocator constructors

2016-04-14 Thread Eric Fiselier via cfe-commits
EricWF updated the summary for this revision. EricWF updated this revision to Diff 53811. http://reviews.llvm.org/D19006 Files: include/tuple test/std/utilities/tuple/tuple.tuple/tuple.cnstr/alloc.pass.cpp test/std/utilities/tuple/tuple.tuple/tuple.cnstr/alloc_UTypes.pass.cpp Index: test/s

Re: [PATCH] D19006: [libcxx] Remove the "reduced-arity-initialization" extension from the uses-allocator constructors

2016-04-14 Thread Eric Fiselier via cfe-commits
EricWF updated this revision to Diff 53813. http://reviews.llvm.org/D19006 Files: include/tuple test/std/utilities/tuple/tuple.tuple/tuple.cnstr/alloc.pass.cpp test/std/utilities/tuple/tuple.tuple/tuple.cnstr/alloc_UTypes.pass.cpp Index: test/std/utilities/tuple/tuple.tuple/tuple.cnstr/all

r266387 - Lit C++11 Compatibility Patch #8

2016-04-14 Thread Charles Li via cfe-commits
Author: lcharles Date: Thu Apr 14 18:47:07 2016 New Revision: 266387 URL: http://llvm.org/viewvc/llvm-project?rev=266387&view=rev Log: Lit C++11 Compatibility Patch #8 24 tests have been updated for C++11 compatibility. Modified: cfe/trunk/test/CXX/basic/basic.lookup/basic.lookup.classref/p1

Re: [PATCH] D19131: Lit C++11 Compatibility Patch #8

2016-04-14 Thread Charles Li via cfe-commits
tigerleapgorge closed this revision. tigerleapgorge added a comment. Committed to Trunk in r266387 http://reviews.llvm.org/D19131 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Re: [PATCH] D18271: Avoid -Wshadow warnings about constructor parameters named after fields

2016-04-14 Thread Richard Smith via cfe-commits
rsmith added inline comments. Comment at: lib/Sema/SemaDecl.cpp:6417-6425 @@ +6416,11 @@ +if (isa(NewDC) && isa(D)) { + if (Diags.isIgnored(diag::warn_ctor_parm_shadows_field, R.getNameLoc())) { +D = D->getCanonicalDecl(); +ShadowingDecls.insert({D, FD});

r266395 - clang-cl: Make /link accept an optional joined argument.

2016-04-14 Thread Hans Wennborg via cfe-commits
Author: hans Date: Thu Apr 14 19:24:15 2016 New Revision: 266395 URL: http://llvm.org/viewvc/llvm-project?rev=266395&view=rev Log: clang-cl: Make /link accept an optional joined argument. For example, "cl.exe a.c /linkfoo bar" is a valid invocation and forwards "foo" and "bar" to link.exe. This m

Re: r266186 - Enable support for __float128 in Clang

2016-04-14 Thread Hans Wennborg via cfe-commits
On Wed, Apr 13, 2016 at 2:49 AM, Nemanja Ivanovic via cfe-commits wrote: > Author: nemanjai > Date: Wed Apr 13 04:49:45 2016 > New Revision: 266186 > > URL: http://llvm.org/viewvc/llvm-project?rev=266186&view=rev > Log: > Enable support for __float128 in Clang > > This patch corresponds to review:

r266402 - clang-cl: Don't check for existence of linker inputs when /link is used

2016-04-14 Thread Hans Wennborg via cfe-commits
Author: hans Date: Thu Apr 14 20:12:32 2016 New Revision: 266402 URL: http://llvm.org/viewvc/llvm-project?rev=266402&view=rev Log: clang-cl: Don't check for existence of linker inputs when /link is used There might be flags passed to the linker (e.g. /libpath), causing it to search in paths the C

[PATCH] D19144: Handle TemplateArgument in DynTypedNode comparison operations.

2016-04-14 Thread Alexander Kornienko via cfe-commits
alexfh created this revision. alexfh added reviewers: sbenza, bkramer. alexfh added a subscriber: cfe-commits. This patch fixes an assertion failure on some matchers containing templateArgument() on some code in some configurations. I couldn't come up with a consistent repro, since the failure de

[PATCH] D19146: [clang-tidy] New checker to detect suspicious string constructor.

2016-04-14 Thread Etienne Bergeron via cfe-commits
etienneb created this revision. etienneb added a reviewer: alexfh. etienneb added a subscriber: cfe-commits. Checker to validate string constructor parameters. A common mistake is to swap parameter for the fill-constructor. ``` std::string str('x', 4); std::string str('4', x); ``` http://rev

[libcxx] r266409 - [libcxx] Remove the "reduced-arity-initialization" extension from the uses-allocator constructors

2016-04-14 Thread Eric Fiselier via cfe-commits
Author: ericwf Date: Thu Apr 14 22:29:40 2016 New Revision: 266409 URL: http://llvm.org/viewvc/llvm-project?rev=266409&view=rev Log: [libcxx] Remove the "reduced-arity-initialization" extension from the uses-allocator constructors Summary: A default uses-allocator constructor has been added sinc

Re: [PATCH] D19006: [libcxx] Remove the "reduced-arity-initialization" extension from the uses-allocator constructors

2016-04-14 Thread Eric Fiselier via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rL266409: [libcxx] Remove the "reduced-arity-initialization" extension from the uses… (authored by EricWF). Changed prior to commit: http://reviews.llvm.org/D19006?vs=53813&id=53835#toc Repository: rL

Re: r266186 - Enable support for __float128 in Clang

2016-04-14 Thread Hal Finkel via cfe-commits
- Original Message - > From: "Hans Wennborg via cfe-commits" > To: "Nemanja Ivanovic" , "Nico Weber" > > Cc: "cfe-commits" > Sent: Thursday, April 14, 2016 8:07:58 PM > Subject: Re: r266186 - Enable support for __float128 in Clang > > On Wed, Apr 13, 2016 at 2:49 AM, Nemanja Ivanovic v

<    1   2