Re: [PATCH] D13844: [libclang] Visit TypeAliasTemplateDecl

2015-11-03 Thread Sergey Kalinichev via cfe-commits
skalinichev added a comment. Yes, I have commit acces now (didn't test it yet though). I'll try to submit this patch soon. Thanks! http://reviews.llvm.org/D13844 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin

Re: [PATCH] D13388: Add support for querying the visibility of a cursor

2015-11-03 Thread Sergey Kalinichev via cfe-commits
skalinichev added a comment. Well, I think it's ok then. Still there is no test for visibility set from command line case. Also adding comment to clang_getCursorVisibility explaining that it also works with visibility set from command line would be very useful. Comment at: te

Re: [PATCH] D10834: Added functions to retrieve information about whether a vardecl is local in libclang and its python bindings.

2015-11-03 Thread guibufolo+l...@gmail.com via cfe-commits
RedX2501 retitled this revision from "Added functions to retrieve information about variable storage in libclang and its python bindings." to "Added functions to retrieve information about whether a vardecl is local in libclang and its python bindings.". RedX2501 updated this revision to Diff 39

Re: [PATCH] D10833: Retrieve BinaryOperator::getOpcode and BinaryOperator::getOpcodeStr via libclang and its python interface

2015-11-03 Thread guibufolo+l...@gmail.com via cfe-commits
RedX2501 updated this revision to Diff 39158. RedX2501 added a comment. Added module tests to libclang and python. http://reviews.llvm.org/D10833 Files: bindings/python/clang/cindex.py bindings/python/tests/cindex/test_cursor.py include/clang-c/Index.h include/clang/AST/OperationKinds.h

Re: [PATCH] D13786: [Sema] Implement __make_integer_seq

2015-11-03 Thread David Majnemer via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rL252036: [Sema] Implement __make_integer_seq (authored by majnemer). Changed prior to commit: http://reviews.llvm.org/D13786?vs=39153&id=39154#toc Repository: rL LLVM http://reviews.llvm.org/D13786

r252036 - [Sema] Implement __make_integer_seq

2015-11-03 Thread David Majnemer via cfe-commits
Author: majnemer Date: Tue Nov 3 21:40:30 2015 New Revision: 252036 URL: http://llvm.org/viewvc/llvm-project?rev=252036&view=rev Log: [Sema] Implement __make_integer_seq This new builtin template allows for incredibly fast instantiations of templates like std::integer_sequence. Performance numb

Re: [PATCH] D13786: [Sema] Implement __make_integer_seq

2015-11-03 Thread David Majnemer via cfe-commits
majnemer updated this revision to Diff 39153. majnemer added a comment. - Address Richard's latest comments http://reviews.llvm.org/D13786 Files: include/clang/AST/ASTContext.h include/clang/AST/DataRecursiveASTVisitor.h include/clang/AST/DeclTemplate.h include/clang/AST/RecursiveASTVis

Re: [PATCH] D14316: [Concepts] Add diagnostics which fall under [dcl.spec.concept]p1

2015-11-03 Thread Hubert Tong via cfe-commits
hubert.reinterpretcast accepted this revision. hubert.reinterpretcast added a comment. This revision is now accepted and ready to land. LGTM http://reviews.llvm.org/D14316 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org

[PATCH] D14316: [Concepts] Add diagnostics which fall under [dcl.spec.concept]p1

2015-11-03 Thread Nathan Wilson via cfe-commits
nwilson created this revision. nwilson added reviewers: rsmith, hubert.reinterpretcast, aaron.ballman, faisalv. nwilson added a subscriber: cfe-commits. Diagnose when the 'concept' specifier is used on a typedef or function parameter. http://reviews.llvm.org/D14316 Files: lib/Sema/SemaDecl.cp

Re: [PATCH] D13786: [Sema] Implement __make_integer_seq

2015-11-03 Thread Richard Smith via cfe-commits
rsmith accepted this revision. rsmith added a comment. This revision is now accepted and ready to land. Please add a test for serialization/deserialization of this declaration. Otherwise, this looks fine. Comment at: include/clang/Serialization/ASTBitCodes.h:994 @@ -990,3 +993,

Re: [PATCH] D13786: [Sema] Implement __make_integer_seq

2015-11-03 Thread David Majnemer via cfe-commits
majnemer updated this revision to Diff 39145. majnemer added a comment. - Add an llvm_unreachable to ASTDeclReader::VisitBuiltinTemplateDecl - Address Richard's review comments. - Add logic to check [intseq.intseq]p1. - Remove TemplateName::BuiltinTemplate - Rename BuiltinTemplateNameKind to Built

Re: [PATCH] D12473: [clang-tidy] Add old style function check

2015-11-03 Thread Alexander Kornienko via cfe-commits
alexfh added a comment. Apparently, I forgot to submit the comments a looong time ago. Sorry for the delay. In http://reviews.llvm.org/D12473#236401, @alexfh wrote: > A high-level comment: > > It seems that the scope of the check is artificially made too narrow. The > check could actually look

Re: [PATCH] D13925: Implement __attribute__((internal_linkage))

2015-11-03 Thread Evgeniy Stepanov via cfe-commits
eugenis added inline comments. Comment at: lib/AST/Decl.cpp:635-641 @@ -634,2 +634,9 @@ assert(!isa(D) && "Didn't expect a FieldDecl!"); + for (const DeclContext *DC = D->getDeclContext(); + !isa(DC); DC = DC->getParent()) { +const NamespaceDecl *ND = dyn_cast(DC);

Re: [PATCH] D13925: Implement __attribute__((internal_linkage))

2015-11-03 Thread Aaron Ballman via cfe-commits
On Tue, Nov 3, 2015 at 7:44 PM, Richard Smith wrote: > On Tue, Nov 3, 2015 at 4:37 PM, Aaron Ballman via cfe-commits > wrote: >> >> On Tue, Nov 3, 2015 at 7:23 PM, Richard Smith >> wrote: >> > rsmith added inline comments. >> > >> > >> > Comment at: include/clang/Basic/AttrDocs.

Re: [PATCH] D13925: Implement __attribute__((internal_linkage))

2015-11-03 Thread Evgeniy Stepanov via cfe-commits
eugenis updated this revision to Diff 39135. eugenis marked 2 inline comments as done. Repository: rL LLVM http://reviews.llvm.org/D13925 Files: include/clang/Basic/Attr.td include/clang/Basic/AttrDocs.td include/clang/Sema/Sema.h lib/AST/Decl.cpp lib/Sema/SemaDecl.cpp lib/Sema/Sem

[PATCH] D14311: [Clang] Fix some Clang-tidy modernize warnings, other minor fixes

2015-11-03 Thread Eugene Zelenko via cfe-commits
Eugene.Zelenko created this revision. Eugene.Zelenko added a reviewer: hans. Eugene.Zelenko added a subscriber: cfe-commits. Eugene.Zelenko set the repository for this revision to rL LLVM. Herald added subscribers: dschuff, jfb. Fixed warnings are: modernize-use-override, modernize-use-nullptr and

Re: [PATCH] D13925: Implement __attribute__((internal_linkage))

2015-11-03 Thread Richard Smith via cfe-commits
On Tue, Nov 3, 2015 at 4:37 PM, Aaron Ballman via cfe-commits < cfe-commits@lists.llvm.org> wrote: > On Tue, Nov 3, 2015 at 7:23 PM, Richard Smith > wrote: > > rsmith added inline comments. > > > > > > Comment at: include/clang/Basic/AttrDocs.td:1628 > > @@ +1627,3 @@ > > +The ``

Re: [PATCH] D13925: Implement __attribute__((internal_linkage))

2015-11-03 Thread Aaron Ballman via cfe-commits
On Tue, Nov 3, 2015 at 7:23 PM, Richard Smith wrote: > rsmith added inline comments. > > > Comment at: include/clang/Basic/AttrDocs.td:1628 > @@ +1627,3 @@ > +The ``internal_linkage`` attribute changes the linkage type of the > declaration to internal. > +This is similar to C-sty

Re: [PATCH] D14179: Remove some legacy mingw-w64 gcc struct info

2015-11-03 Thread Martell Malone via cfe-commits
martell added a comment. From what I can gather it made sense at the time of gcc 4.6 so I don't think anyone is at fault :) Repository: rL LLVM http://reviews.llvm.org/D14179 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.l

Re: [PATCH] D13925: Implement __attribute__((internal_linkage))

2015-11-03 Thread Richard Smith via cfe-commits
rsmith added inline comments. Comment at: include/clang/Basic/AttrDocs.td:1628 @@ +1627,3 @@ +The ``internal_linkage`` attribute changes the linkage type of the declaration to internal. +This is similar to C-style ``static``, but can be used on classes and class methods +This ca

Re: [PATCH] D13304: Avoid inlining in throw statement

2015-11-03 Thread Manman Ren via cfe-commits
manmanren added a subscriber: manmanren. manmanren added a comment. Inliner currently does not include analysis passes such as BPI and BFI. With the new pass manager, we should be able to hook up inliner with BFI (and we can handle throw in BFI). Before that, maybe we can add this as part of inl

Re: [PATCH] D13925: Implement __attribute__((internal_linkage))

2015-11-03 Thread Evgeniy Stepanov via cfe-commits
eugenis updated this revision to Diff 39128. eugenis added a comment. One more test for forward declarations. Repository: rL LLVM http://reviews.llvm.org/D13925 Files: include/clang/Basic/Attr.td include/clang/Basic/AttrDocs.td include/clang/Sema/Sema.h lib/AST/Decl.cpp lib/Sema/Se

Re: [PATCH] D14277: [Analyzer] Make referenced SymbolMetadata live even if its region is dead

2015-11-03 Thread Jordan Rose via cfe-commits
jordan_rose added a comment. The intent here was that a metadata symbol represents //metadata about a region.// That means if the region is dead, and the symbol isn't directly referenceable, we won't be able to recreate it. The advantage of `&&` was that checkers didn't have to track when regio

Re: [PATCH] D13304: Avoid inlining in throw statement

2015-11-03 Thread Jun Bum Lim via cfe-commits
junbuml added a comment. Thanks Richard for your comment ! If the frond-end is not a good to place for this, I think there are two places we can consider : inliner or prune-eh. 1. In inliner, we can directly check if a CallSite branches an exception region, and then make getInlineThreshold() r

Re: [PATCH] D13925: Implement __attribute__((internal_linkage))

2015-11-03 Thread Evgeniy Stepanov via cfe-commits
eugenis added inline comments. Comment at: include/clang/Basic/Attr.td:2125 @@ +2124,3 @@ + +def InternalLinkage : InheritableAttr { + let Spellings = [GNU<"internal_linkage">, CXX11<"clang", "internal_linkage">]; aaron.ballman wrote: > rsmith wrote: > > `Inheri

Re: [PATCH] D13925: Implement __attribute__((internal_linkage))

2015-11-03 Thread Evgeniy Stepanov via cfe-commits
eugenis updated this revision to Diff 39124. eugenis marked 5 inline comments as done. eugenis added a comment. Disabled the new attribute on namespaces. Repository: rL LLVM http://reviews.llvm.org/D13925 Files: include/clang/Basic/Attr.td include/clang/Basic/AttrDocs.td include/clang/

Re: [PATCH] D13980: Add "x87" in x86 target feature map

2015-11-03 Thread Richard Smith via cfe-commits
rsmith added inline comments. Comment at: lib/Basic/Targets.cpp:2516-2517 @@ -2515,1 +2515,4 @@ + // X86 always has X87. + setFeatureEnabledImpl(Features, "x87", true); + i386 doesn't necessarily have x87, does it? IIRC the floating point co-processor is only

[PATCH] D14307: AMDGPU: Add support for 's' and 'v' asm constraints

2015-11-03 Thread Tom Stellard via cfe-commits
tstellarAMD created this revision. tstellarAMD added reviewers: echristo, arsenm. tstellarAMD added a subscriber: cfe-commits. 's' is used to specify sgprs and 'v' is used to specify vgprs. http://reviews.llvm.org/D14307 Files: lib/Basic/Targets.cpp test/Sema/inline-asm-validate-amdgpu.cl I

Re: [PATCH] D14164: Driver: fallback to parent directory of clang if no sysroot for mingw-w64 target

2015-11-03 Thread Yaron Keren via cfe-commits
yaron.keren added a subscriber: cfe-commits. yaron.keren added a comment. We did not have cfe-commits as subscriber so I'm adding it now (this is a clang commit), see if someone would like to further comment. http://reviews.llvm.org/D14164 ___ cfe

Re: [PATCH] D14179: Remove some legacy mingw-w64 gcc struct info

2015-11-03 Thread NAKAMURA Takumi via cfe-commits
chapuni added a comment. Sorry, I supposed someone removed it. It was really buggy in past. Repository: rL LLVM http://reviews.llvm.org/D14179 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/

Re: [PATCH] D12031: Const std::move() argument ClangTidy check

2015-11-03 Thread Alexander Kornienko via cfe-commits
alexfh added inline comments. Comment at: clang-tidy/misc/MoveConstantArgumentCheck.cpp:30 @@ +29,3 @@ +bool IsVariable = dyn_cast(Arg) != nullptr; +std::string message = "std::move of the "; +message += IsConstArg ? "const " : ""; alexfh wrote: > Plea

Re: [PATCH] D12031: Const std::move() argument ClangTidy check

2015-11-03 Thread Alexander Kornienko via cfe-commits
alexfh added inline comments. Comment at: clang-tidy/misc/MoveConstantArgumentCheck.cpp:22 @@ +21,3 @@ + bool IsTypeDependOnTemplateParameter = + false; // my first guess was type->getTypeClass () == 30 but it doesn't + // work in some cases. Could you please a

[PATCH] D14303: [analyzer] Add 'optin' checker package and move localizability checkers into it.

2015-11-03 Thread Devin Coughlin via cfe-commits
dcoughlin created this revision. dcoughlin added reviewers: zaks.anna, krememek. dcoughlin added a subscriber: cfe-commits. This patch creates a new 'optin' top-level checker package and moves several of the localizability checkers into it. This package is for checkers that are not alpha and th

Re: [PATCH] D14293: [libcxx] Add -fno-exceptions libcxx builders to zorg

2015-11-03 Thread Asiri Rathnayake via cfe-commits
rmaprath added inline comments. Comment at: buildbot/osuosl/master/config/builders.py:874 @@ -842,1 +873,3 @@ 'CMAKE_CXX_FLAGS': '-mcpu=cortex-a15'})}, + +# Cortex-A15 -fno-exceptions libcxx build (skips libcxx tests) rengoli

Re: [PATCH] D13304: Avoid inlining in throw statement

2015-11-03 Thread Richard Smith via cfe-commits
rsmith added a comment. I am not convinced that it's reasonable to put inlining heuristics into clang's IR generation. This will cause maintenance problems for the inliner in the future (anyone tuning the inlining heuristics and thresholds will need to be aware of this, and clang will behave di

Re: [PATCH] D14293: [libcxx] Add -fno-exceptions libcxx builders to zorg

2015-11-03 Thread Asiri Rathnayake via cfe-commits
rmaprath added inline comments. Comment at: buildbot/osuosl/master/config/builders.py:771 @@ +770,3 @@ + # the -fno-exceptions library build. + additional_features={'no-libcxx-tests'}), + 'category': 'libcxx'}, jroelofs wrote: > How

Re: [PATCH] D13304: Avoid inlining in throw statement

2015-11-03 Thread Chad Rosier via cfe-commits
mcrosier added a comment. @chandlerc: Adding Chandler in case he has an opinion on how to move forward or how we could go about tuning the cold threshold. http://reviews.llvm.org/D13304 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http:/

Re: [PATCH] D14184: [clang] Add initial support for -meabi flag

2015-11-03 Thread Vinicius Tinti via cfe-commits
tinti set the repository for this revision to rL LLVM. tinti updated this revision to Diff 39108. tinti added a comment. - Clang format code. - Update eabi names to match new LLVM patch. Repository: rL LLVM http://reviews.llvm.org/D14184 Files: include/clang/Driver/Options.td include/cla

Re: [PATCH] D13099: [Analyzer] Don’t invalidate CXXThis when conservatively evaluating const methods (PR 21606)

2015-11-03 Thread Devin Coughlin via cfe-commits
dcoughlin added a comment. This patch seems to have caused a regression. https://llvm.org/bugs/show_bug.cgi?id=25392 Repository: rL LLVM http://reviews.llvm.org/D13099 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org

Re: [PATCH] D14293: [libcxx] Add -fno-exceptions libcxx builders to zorg

2015-11-03 Thread Jonathan Roelofs via cfe-commits
jroelofs added a subscriber: jroelofs. Comment at: buildbot/osuosl/master/config/builders.py:771 @@ +770,3 @@ + # the -fno-exceptions library build. + additional_features={'no-libcxx-tests'}), + 'category': 'libcxx'}, How bad is the

Re: [PATCH] D14293: [libcxx] Add -fno-exceptions libcxx builders to zorg

2015-11-03 Thread Renato Golin via cfe-commits
rengolin added inline comments. Comment at: buildbot/osuosl/master/config/builders.py:874 @@ -842,1 +873,3 @@ 'CMAKE_CXX_FLAGS': '-mcpu=cortex-a15'})}, + +# Cortex-A15 -fno-exceptions libcxx build (skips libcxx tests) I'd lik

r251966 - [analyzer] Move the ObjCGenericsChecker out of the alpha package.

2015-11-03 Thread Devin Coughlin via cfe-commits
Author: dcoughlin Date: Tue Nov 3 13:38:03 2015 New Revision: 251966 URL: http://llvm.org/viewvc/llvm-project?rev=251966&view=rev Log: [analyzer] Move the ObjCGenericsChecker out of the alpha package. It is now in the osx.cocoa package and so will be on by default for Apple toolchains. Modified

Re: [PATCH] D14291: Improve modernize-make-unique matcher.

2015-11-03 Thread Manuel Klimek via cfe-commits
klimek accepted this revision. klimek added a comment. This revision is now accepted and ready to land. lg http://reviews.llvm.org/D14291 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-comm

Re: [PATCH] D13304: Avoid inlining in throw statement

2015-11-03 Thread Jun Bum Lim via cfe-commits
junbuml added a comment. The basic idea of this change is to avoid inlining callsites invoked in exception handling regions (EHR) so that we can reduce code size blow-up in very cold regions and indirectly increase inline opportunities for functions containing exception handling code. I think

Re: [PATCH] D13446: [PATCH] Add checker discouraging definition of variadic function definitions in C++

2015-11-03 Thread Daniel Berlin via cfe-commits
Apologies, I will try to take a look today On Tue, Nov 3, 2015, 10:05 AM Aaron Ballman wrote: > On Tue, Nov 3, 2015 at 7:19 AM, Alexander Kornienko > wrote: > > On Fri, Oct 9, 2015 at 12:13 PM, Aaron Ballman > > wrote: > >> > >> On Fri, Oct 9, 2015 at 3:09 PM, Daniel Berlin > wrote: > >> > db

Re: [PATCH] D14293: [libcxx] Add -fno-exceptions libcxx builders to zorg

2015-11-03 Thread Dmitri Gribenko via cfe-commits
gribozavr added a comment. The Debian configuration LGTM. http://reviews.llvm.org/D14293 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

r251955 - Introduce module file extensions to piggy-back data onto module files.

2015-11-03 Thread Douglas Gregor via cfe-commits
Author: dgregor Date: Tue Nov 3 12:33:07 2015 New Revision: 251955 URL: http://llvm.org/viewvc/llvm-project?rev=251955&view=rev Log: Introduce module file extensions to piggy-back data onto module files. Introduce the notion of a module file extension, which introduces additional information int

Re: [PATCH] D14293: [libcxx] Add -fno-exceptions libcxx builders to zorg

2015-11-03 Thread Asiri Rathnayake via cfe-commits
rmaprath added a comment. PS: I've run a couple of hand-crafted builders on a locally hacked up buildbot and made sure the above changes are fine. http://reviews.llvm.org/D14293 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.l

Re: [PATCH] D12031: Const std::move() argument ClangTidy check

2015-11-03 Thread Aaron Ballman via cfe-commits
aaron.ballman added a subscriber: aaron.ballman. Comment at: clang-tidy/misc/MoveConstantArgumentCheck.cpp:1 @@ +1,2 @@ +#include "MoveConstantArgumentCheck.h" + Missing new file legal text. Comment at: clang-tidy/misc/MoveConstantArgumentCheck.c

[PATCH] D14293: [libcxx] Add -fno-exceptions libcxx builders to zorg

2015-11-03 Thread Asiri Rathnayake via cfe-commits
rmaprath created this revision. rmaprath added reviewers: rengolin, gribozavr, gkistanova, EricWF. rmaprath added a subscriber: cfe-commits. This patch adds the necessary builders to zorg to get the -fno-exceptions libcxx library variant building. I've only used those build-slaves which I have g

r251953 - Revert "Support watchOS and tvOS in compiler-rt builds"

2015-11-03 Thread Tim Northover via cfe-commits
Author: tnorthover Date: Tue Nov 3 12:14:42 2015 New Revision: 251953 URL: http://llvm.org/viewvc/llvm-project?rev=251953&view=rev Log: Revert "Support watchOS and tvOS in compiler-rt builds" The required compiler-rt changes aren't present yet so attempting to build with compiler-rt breaks. And

[PATCH] D14292: [libcxx] Make it possible to build a no-exceptions variant of libcxx.

2015-11-03 Thread Asiri Rathnayake via cfe-commits
rmaprath created this revision. rmaprath added reviewers: rengolin, mclow.lists. rmaprath added a subscriber: cfe-commits. Fixes a small omission in libcxx that prevents libcxx being built when -DLIBCXX_ENABLE_EXCEPTIONS=0 is used. I will do a follow-up patch to introduce the new libc++ build to

Re: [PATCH] D14172: [libcxx] Make it possible to build a no-exceptions variant of libcxx.

2015-11-03 Thread Asiri Rathnayake via cfe-commits
rmaprath abandoned this revision. rmaprath added a comment. Abandoning revision as it seems I cannot add subscribers/reviewers to existing phab reviews. http://reviews.llvm.org/D14172 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://l

Re: [PATCH] D12031: Const std::move() argument ClangTidy check

2015-11-03 Thread Vadym Doroshenko via cfe-commits
dvadym updated the summary for this revision. dvadym updated this revision to Diff 39077. dvadym marked 6 inline comments as done. dvadym added a comment. 1.Most comments addressed 2.Taking into consideration applying move to trivially copyable objects 3.Different message if move argument variable

Re: [PATCH] D13446: [PATCH] Add checker discouraging definition of variadic function definitions in C++

2015-11-03 Thread Aaron Ballman via cfe-commits
On Tue, Nov 3, 2015 at 7:19 AM, Alexander Kornienko wrote: > On Fri, Oct 9, 2015 at 12:13 PM, Aaron Ballman > wrote: >> >> On Fri, Oct 9, 2015 at 3:09 PM, Daniel Berlin wrote: >> > dberlin added a subscriber: dberlin. >> > >> > >> > Comment at: docs/clang-tidy/checks/cert-variad

Re: [PATCH] D12031: Const std::move() argument ClangTidy check

2015-11-03 Thread Vadym Doroshenko via cfe-commits
dvadym updated this revision to Diff 39080. dvadym marked an inline comment as done. dvadym added a comment. Small clean-up http://reviews.llvm.org/D12031 Files: clang-tidy/misc/CMakeLists.txt clang-tidy/misc/MiscTidyModule.cpp clang-tidy/misc/MoveConstantArgumentCheck.cpp clang-tidy/mi

Re: [PATCH] D12031: Const std::move() argument ClangTidy check

2015-11-03 Thread Vadym Doroshenko via cfe-commits
dvadym added a comment. Thanks alexfh and sbenza for comments! I've addressed most of them. Could you please advice how to find that some expression has type which is template dependent? Regards, Vadym Comment at: clang-tidy/misc/MoveConstantArgumentCheck.cpp:20 @@ +19,3 @@ +

Re: [PATCH] D14180: enable -fms-extensions by default on the mingw-w64 target

2015-11-03 Thread Martell Malone via cfe-commits
martell added a comment. In http://reviews.llvm.org/D14180#280184, @rnk wrote: > You're mostly interested in the non-builtin parts of -fms-extensions, like > declspec and UUID, right? Yes that would be one of the main reasons, gcc is very lacking in this department :) > You can then change t

[PATCH] D14291: Improve modernize-make-unique matcher.

2015-11-03 Thread Angel Garcia via cfe-commits
angelgarcia created this revision. angelgarcia added a reviewer: klimek. angelgarcia added subscribers: cfe-commits, alexfh. "std::unique_ptr" is not the same type as "std::unique_ptr>", unless we insert a "hasCanonicalType" in the middle. Probably it also happens in other cases related to defaul

Re: [PATCH] D14180: enable -fms-extensions by default on the mingw-w64 target

2015-11-03 Thread Reid Kleckner via cfe-commits
rnk added a comment. I'd like clang to work out of the box with mingw-w64 releases from at least the past year. You're mostly interested in the non-builtin parts of -fms-extensions, like declspec and UUID, right? One way we could do this is to enable -fms-extensions in mingw but then suppress t

Re: [PATCH] D14180: enable -fms-extensions by default on the mingw-w64 target

2015-11-03 Thread Martell Malone via cfe-commits
martell added a comment. > This is what I'm worried about. :) Can you elaborate on what would break? Essentially mingw-w64 has a few header defines for working around builtings such as __rtdsc and the interlocked functions and some other builtins As soon as one of these headers are included it

Re: [PATCH] D14266: CodeGen: Update for debug info API change.Depends on D14265.

2015-11-03 Thread Duncan P. N. Exon Smith via cfe-commits
This LGTM (although obviously gated on LLVM changes). > On 2015-Nov-02, at 16:28, Peter Collingbourne wrote: > > pcc created this revision. > pcc added a reviewer: dexonsmith. > pcc added a subscriber: cfe-commits. > > http://reviews.llvm.org/D14266 > > Files: > lib/CodeGen/CGDebugInfo.cpp >

Re: [PATCH] D14180: enable -fms-extensions by default on the mingw-w64 target

2015-11-03 Thread Reid Kleckner via cfe-commits
rnk added a comment. In http://reviews.llvm.org/D14180#279962, @martell wrote: > In http://reviews.llvm.org/D14180#279651, @rnk wrote: > > > Will Clang be able to compile the last few stable mingw-w64 releases with > > this change? I'd like that to keep working. > > > Clang has not been able to

Re: [PATCH] D14285: [x86] Additional small fix for MCU psABI support

2015-11-03 Thread Reid Kleckner via cfe-commits
rnk accepted this revision. rnk added a comment. This revision is now accepted and ready to land. lgtm http://reviews.llvm.org/D14285 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

r251949 - Simplify Sema::ProcessPropertyDecl. NFC

2015-11-03 Thread Douglas Gregor via cfe-commits
Author: dgregor Date: Tue Nov 3 11:02:34 2015 New Revision: 251949 URL: http://llvm.org/viewvc/llvm-project?rev=251949&view=rev Log: Simplify Sema::ProcessPropertyDecl. NFC Now that the properties created within Objective-C class extensions go into the extension themselves, we don't need any of

[clang-tools-extra] r251943 - Handle correctly containers that are data members in modernize-loop-convert.

2015-11-03 Thread Angel Garcia Gomez via cfe-commits
Author: angelgarcia Date: Tue Nov 3 10:38:31 2015 New Revision: 251943 URL: http://llvm.org/viewvc/llvm-project?rev=251943&view=rev Log: Handle correctly containers that are data members in modernize-loop-convert. Summary: I recently found that the variable naming wasn't working as expected with

Re: [PATCH] D14289: Handle correctly containers that are data members in modernize-loop-convert.

2015-11-03 Thread Angel Garcia via cfe-commits
angelgarcia updated this revision to Diff 39075. angelgarcia added a comment. Merge with the latest revision. http://reviews.llvm.org/D14289 Files: clang-tidy/modernize/LoopConvertCheck.cpp clang-tidy/modernize/LoopConvertCheck.h clang-tidy/modernize/LoopConvertUtils.h test/clang-tidy/m

r251941 - Remove unused #include

2015-11-03 Thread David Majnemer via cfe-commits
Author: majnemer Date: Tue Nov 3 10:32:15 2015 New Revision: 251941 URL: http://llvm.org/viewvc/llvm-project?rev=251941&view=rev Log: Remove unused #include No functionality change is intended. Modified: cfe/trunk/lib/CodeGen/CGExprCXX.cpp Modified: cfe/trunk/lib/CodeGen/CGExprCXX.cpp URL:

Re: [PATCH] D14289: Handle correctly containers that are data members in modernize-loop-convert.

2015-11-03 Thread Manuel Klimek via cfe-commits
klimek accepted this revision. klimek added a comment. This revision is now accepted and ready to land. lg http://reviews.llvm.org/D14289 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-comm

[clang-tools-extra] r251940 - Improve more the const-detection in modernize-loop-convert.

2015-11-03 Thread Angel Garcia Gomez via cfe-commits
Author: angelgarcia Date: Tue Nov 3 10:31:36 2015 New Revision: 251940 URL: http://llvm.org/viewvc/llvm-project?rev=251940&view=rev Log: Improve more the const-detection in modernize-loop-convert. Summary: The previous change was focused in detecting when a non-const object was used in a consta

Re: [PATCH] D14282: Improve more the const-detection in modernize-loop-convert.

2015-11-03 Thread Angel Garcia via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rL251940: Improve more the const-detection in modernize-loop-convert. (authored by angelgarcia). Changed prior to commit: http://reviews.llvm.org/D14282?vs=39052&id=39074#toc Repository: rL LLVM http:

Re: [PATCH] D14282: Improve more the const-detection in modernize-loop-convert.

2015-11-03 Thread Manuel Klimek via cfe-commits
klimek accepted this revision. klimek added a comment. This revision is now accepted and ready to land. lg http://reviews.llvm.org/D14282 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-comm

[PATCH] D14286: ASTImporter: expressions, pt.1

2015-11-03 Thread Aleksei Sidorin via cfe-commits
a.sidorin created this revision. a.sidorin added a reviewer: sepavloff. a.sidorin added a subscriber: cfe-commits. a.sidorin set the repository for this revision to rL LLVM. This patch implements some expression-related AST node import. This is the first patch in a series. Supported nodes: GCCAs

[PATCH] D14289: Handle correctly containers that are data members in modernize-loop-convert.

2015-11-03 Thread Angel Garcia via cfe-commits
angelgarcia created this revision. angelgarcia added a reviewer: klimek. angelgarcia added subscribers: alexfh, cfe-commits. I recently found that the variable naming wasn't working as expected with containers that are data members. The new index always received the name "Elem" (or equivalent) r

Re: [PATCH] D14179: Remove some legacy mingw-w64 gcc struct info

2015-11-03 Thread Martell Malone via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rL251930: Remove some legacy mingw-w64 gcc struct info (authored by martell). Changed prior to commit: http://reviews.llvm.org/D14179?vs=38744&id=39067#toc Repository: rL LLVM http://reviews.llvm.org/

r251930 - Remove some legacy mingw-w64 gcc struct info

2015-11-03 Thread Martell Malone via cfe-commits
Author: martell Date: Tue Nov 3 09:57:45 2015 New Revision: 251930 URL: http://llvm.org/viewvc/llvm-project?rev=251930&view=rev Log: Remove some legacy mingw-w64 gcc struct info As of gcc 4.7 mingw-w64 no longer emits 128-bit structs as i128 Differential Revision: http://reviews.llvm.org/D14179

[PATCH] D14285: [x86] Additional small fix for MCU psABI support

2015-11-03 Thread Andrey Bokhanko via cfe-commits
andreybokhanko created this revision. andreybokhanko added reviewers: rnk, DavidKreitzer, mkuper. andreybokhanko added a subscriber: cfe-commits. This patch fixes one more thing in MCU psABI support: LongDoubleWidth should be set to 64. http://reviews.llvm.org/D14285 Files: lib/Basic/Targets.

[PATCH] D14282: Improve more the const-detection in modernize-loop-convert.

2015-11-03 Thread Angel Garcia via cfe-commits
angelgarcia created this revision. angelgarcia added a reviewer: klimek. angelgarcia added subscribers: cfe-commits, alexfh. The previous change was focused in detecting when a non-const object was used in a constant way. Looks like I forgot the most important and trivial case: when the object i

Re: [PATCH] D14180: enable -fms-extensions by default on the mingw-w64 target

2015-11-03 Thread Martell Malone via cfe-commits
martell added a comment. In http://reviews.llvm.org/D14180#279651, @rnk wrote: > Will Clang be able to compile the last few stable mingw-w64 releases with > this change? I'd like that to keep working. Clang has not been able to compile mingw-w64 crt for the last few releases. I have only added

Re: [PATCH] D14184: [clang] Add initial support for -meabi flag

2015-11-03 Thread Renato Golin via cfe-commits
rengolin added inline comments. Comment at: lib/Frontend/CompilerInvocation.cpp:458 @@ +457,3 @@ +StringRef Value = A->getValue(); +bool Valid = llvm::StringSwitch(Value) + .Case("default", true) tinti wrote: > tinti wrote: > > compnerd wrote: > > > I

Re: [PATCH] D13446: [PATCH] Add checker discouraging definition of variadic function definitions in C++

2015-11-03 Thread Alexander Kornienko via cfe-commits
On Fri, Oct 9, 2015 at 12:13 PM, Aaron Ballman wrote: > On Fri, Oct 9, 2015 at 3:09 PM, Daniel Berlin wrote: > > dberlin added a subscriber: dberlin. > > > > > > Comment at: docs/clang-tidy/checks/cert-variadic-function-def.rst:13 > > @@ +12,2 @@ > > +`DCL50-CPP. Do not define a

Re: [PATCH] D14014: Checker of proper vfork usage

2015-11-03 Thread Yury Gribov via cfe-commits
ygribov marked 2 inline comments as done. ygribov added a comment. http://reviews.llvm.org/D14014 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Re: [PATCH] D14014: Checker of proper vfork usage

2015-11-03 Thread Yury Gribov via cfe-commits
ygribov updated this revision to Diff 39040. ygribov added a comment. Updated warning messages. http://reviews.llvm.org/D14014 Files: include/clang/StaticAnalyzer/Core/PathSensitive/CheckerHelpers.h lib/StaticAnalyzer/Checkers/CMakeLists.txt lib/StaticAnalyzer/Checkers/Checkers.td lib/S