[PATCH] D30214: [Driver] Search for libc++ headers in ResourceDir

Hahnfeld added a comment. In https://reviews.llvm.org/D30214#690010, @jroelofs wrote: > libc++ headers should not be installed in the resource dir. They are currently not by default. But with https://reviews.llvm.org/D30015 for runtime libraries, this may become a place for headers packaged wi

[PATCH] D30517: [libc++abi] Add option to enable definitions for the new/delete overloads.

mehdi_amini accepted this revision. mehdi_amini added a comment. This revision is now accepted and ready to land. LGTM. Comment at: CMakeLists.txt:416 +set(LIBCXXABI_HAS_UNDEFINED_SYMBOLS ((NOT LIBCXXABI_ENABLE_NEW_DELETE_DEFINITIONS) +OR (LIBCXXABI_BUILD_EXTERNAL_THREAD_L

[PATCH] D30516: [libc++] Add option to disable new/delete overloads when libc++abi provides them.

mehdi_amini accepted this revision. mehdi_amini added a comment. This revision is now accepted and ready to land. LGTM https://reviews.llvm.org/D30516 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/list

[PATCH] D30514: [libc++abi] Update new/delete definitions to match libc++

mehdi_amini accepted this revision. mehdi_amini added a comment. This revision is now accepted and ready to land. LGTM. (It seems that having libc++ and libc++abi in the same repo would help sharing code like this) https://reviews.llvm.org/D30514

[libcxx] r296732 - [libc++] Make _LIBCPP_TYPE_VIS export members

Author: smeenai Date: Wed Mar 1 21:22:18 2017 New Revision: 296732 URL: http://llvm.org/viewvc/llvm-project?rev=296732&view=rev Log: [libc++] Make _LIBCPP_TYPE_VIS export members Summary: Most classes annotated with _LIBCPP_TYPE_VIS need to have at least some of their members exported, otherwise

[PATCH] D25208: [libc++] Make _LIBCPP_TYPE_VIS export members

This revision was automatically updated to reflect the committed changes. Closed by commit rL296732: [libc++] Make _LIBCPP_TYPE_VIS export members (authored by smeenai). Changed prior to commit: https://reviews.llvm.org/D25208?vs=90275&id=90276#toc Repository: rL LLVM https://reviews.llvm.o

[PATCH] D25208: [libc++] Make _LIBCPP_TYPE_VIS export members

smeenai updated this revision to Diff 90275. smeenai added a comment. Rebase https://reviews.llvm.org/D25208 Files: docs/DesignDocs/VisibilityMacros.rst include/__config include/__locale include/__mutex_base include/condition_variable include/future include/mutex include/shared_

[PATCH] D29157: [libc++] Make _LIBCPP_EXTERN_TEMPLATE_TYPE_VIS export members

This revision was automatically updated to reflect the committed changes. Closed by commit rL296731: [libc++] Make _LIBCPP_EXTERN_TEMPLATE_TYPE_VIS export members (authored by smeenai). Changed prior to commit: https://reviews.llvm.org/D29157?vs=90272&id=90274#toc Repository: rL LLVM https:

[libcxx] r296731 - [libc++] Make _LIBCPP_EXTERN_TEMPLATE_TYPE_VIS export members

Author: smeenai Date: Wed Mar 1 21:02:50 2017 New Revision: 296731 URL: http://llvm.org/viewvc/llvm-project?rev=296731&view=rev Log: [libc++] Make _LIBCPP_EXTERN_TEMPLATE_TYPE_VIS export members When building libc++ with hidden visibility, we want explicit template instantiations to export membe

[PATCH] D29157: [libc++] Make _LIBCPP_EXTERN_TEMPLATE_TYPE_VIS export members

smeenai updated this revision to Diff 90272. smeenai added a comment. Including inline in macro to work around PR32114 https://reviews.llvm.org/D29157 Files: docs/DesignDocs/VisibilityMacros.rst include/__config include/locale include/string Index: include/string ==

[PATCH] D30523: [libc++] Mark some std::num_get method templates inline

This revision was automatically updated to reflect the committed changes. Closed by commit rL296729: [libc++] Mark some std::num_get method templates inline (authored by smeenai). Changed prior to commit: https://reviews.llvm.org/D30523?vs=90270&id=90271#toc Repository: rL LLVM https://revi

[libcxx] r296729 - [libc++] Mark some std::num_get method templates inline

Author: smeenai Date: Wed Mar 1 20:51:27 2017 New Revision: 296729 URL: http://llvm.org/viewvc/llvm-project?rev=296729&view=rev Log: [libc++] Mark some std::num_get method templates inline D29157 will make explicit template instantiations expand to default visibility, at which point these method

[PATCH] D30523: [libc++] Mark some std::num_get method templates inline

smeenai created this revision. https://reviews.llvm.org/D29157 will make explicit template instantiations expand to default visibility, at which point these method templates will need to be explicitly marked hidden visibility to avoid leaking into other DSOs. Unfortunately, because of clang PR321

[PATCH] D30345: [CodeGen][Blocks] Refactor capture handling in code that generates block copy/destroy routines

rjmccall added a comment. Thanks, LGTM. Repository: rL LLVM https://reviews.llvm.org/D30345 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[libcxx] r296727 - Remove more usages of REQUIRES-ANY in the test-suite

Author: ericwf Date: Wed Mar 1 20:04:49 2017 New Revision: 296727 URL: http://llvm.org/viewvc/llvm-project?rev=296727&view=rev Log: Remove more usages of REQUIRES-ANY in the test-suite Modified: libcxx/trunk/test/std/depr/depr.auto.ptr/auto.ptr/auto.ptr.cons/assignment.pass.cpp libcxx/

[libcxx] r296725 - Recommit r296712: "Fix Apple-specific XFAIL directive in libc++ test"

Author: mehdi_amini Date: Wed Mar 1 20:01:11 2017 New Revision: 296725 URL: http://llvm.org/viewvc/llvm-project?rev=296725&view=rev Log: Recommit r296712: "Fix Apple-specific XFAIL directive in libc++ test" The test is passing with c++11 and c++14 but not c++1z on this particular version of the

[libcxx] r296724 - Revert "Fix Apple-specific XFAIL directive in libc++ test"

Author: mehdi_amini Date: Wed Mar 1 19:57:40 2017 New Revision: 296724 URL: http://llvm.org/viewvc/llvm-project?rev=296724&view=rev Log: Revert "Fix Apple-specific XFAIL directive in libc++ test" This reverts commit r296712. It broke our bot. It turns out that the test is passing with c++11 and

[PATCH] D29157: [libc++] Make _LIBCPP_EXTERN_TEMPLATE_TYPE_VIS export members

smeenai updated this revision to Diff 90264. smeenai added a comment. Changing more instances to new macro, per IRC discussion with Eric https://reviews.llvm.org/D29157 Files: docs/DesignDocs/VisibilityMacros.rst include/__config include/locale include/string Index: include/string

[PATCH] D28348: [analyzer] Taught the analyzer about Glib API to check Memory-leak

xiangzhai added a comment. Hi Anna, Thanks for your review! > The information generated by this checker is used for array bounds checking. > For example, see https://reviews.llvm.org/D24307 I will read carefully about that ;-) > This patch looks good. Do you have commit access or should I com

RE: r296554 - [PS4] Set our default dialect to C++11. NFC for other targets.

Mehdi, again I invite you to discuss this on the cfe-dev thread rather than here. I have posted a reply there to bump it up, and any comments you have will be more visible and better serve the community over there. Thanks, --paulr From: mehdi.am...@apple.com [mailto:mehdi.am...@apple.com] Sent:

[PATCH] D30518: Fix msc-version.c test to handle _MSC_VER=1910

rnk accepted this revision. rnk added a comment. This revision is now accepted and ready to land. lgtm https://reviews.llvm.org/D30518 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[PATCH] D30518: Fix msc-version.c test to handle _MSC_VER=1910

DaveBartolomeo created this revision. Previously, VC++ has always set _MSC_VER to a four-digit value with the two least significant digits set to zero. Visual Studio 2017, however, sets _MSC_VER=1910, and we expect to update the least significant digit as we release major updates for VS 2017. T

[PATCH] D30517: [libc++abi] Add option to enable definitions for the new/delete overloads.

EricWF created this revision. Currently both libc++ and libc++abi provide definitions for operator new/delete. However I believe this is incorrect and that one or the other should offer them. This patch adds the CMake option `-DLIBCXXABI_ENABLE_NEW_DELETE_DEFINITIONS` which defaults to `OFF` u

[PATCH] D30516: [libc++] Add option to disable new/delete overloads when libc++abi provides them.

EricWF created this revision. Currently both libc++ and libc++abi provide definitions for operator new/delete. However I believe this is incorrect and that one or the other should offer them. This patch adds the CMake option `-DLIBCXX_ENABLE_NEW_DELETE_DEFINITIONS` which defaults no `ON` unles

[libcxx] r296716 - Fix python 3 syntax error in sym_diff

Author: ericwf Date: Wed Mar 1 18:03:01 2017 New Revision: 296716 URL: http://llvm.org/viewvc/llvm-project?rev=296716&view=rev Log: Fix python 3 syntax error in sym_diff Modified: libcxx/trunk/utils/libcxx/sym_check/diff.py Modified: libcxx/trunk/utils/libcxx/sym_check/diff.py URL: http://

[libcxxabi] r296715 - Cleanup new/delete definitions

Author: ericwf Date: Wed Mar 1 17:59:34 2017 New Revision: 296715 URL: http://llvm.org/viewvc/llvm-project?rev=296715&view=rev Log: Cleanup new/delete definitions This patch cleans up how libc++abi handles the definitions for new/delete. It is in preperation for upcoming changes to fix how both

[libcxx] r296712 - Fix Apple-specific XFAIL directive in libc++ test

Author: mehdi_amini Date: Wed Mar 1 17:45:03 2017 New Revision: 296712 URL: http://llvm.org/viewvc/llvm-project?rev=296712&view=rev Log: Fix Apple-specific XFAIL directive in libc++ test This tests is failing in XCode 7.0. But Xcode 7.3 that shipped an updated clang has this test passing. This i

Re: r296554 - [PS4] Set our default dialect to C++11. NFC for other targets.

> On Mar 1, 2017, at 3:24 PM, Sean Silva wrote: > > > > On Wed, Mar 1, 2017 at 10:35 AM, Mehdi Amini > wrote: > I’m not sure I find this nice to see this upstream. > > I not fond in general of this kind of difference in behavior. I don’t think > it is good for

[PATCH] D30087: [Driver] Unify linking of OpenMP runtime

hfinkel added inline comments. Comment at: lib/Driver/Tools.cpp:10334 - if (Args.hasFlag(options::OPT_fopenmp, options::OPT_fopenmp_EQ, - options::OPT_fno_openmp, false)) { + // FIXME: Exclude this for platforms with libgomp that don't require +

[libcxx] r296704 - Fix Apple-specific XFAIL directive in libc++ test

Author: mehdi_amini Date: Wed Mar 1 17:14:34 2017 New Revision: 296704 URL: http://llvm.org/viewvc/llvm-project?rev=296704&view=rev Log: Fix Apple-specific XFAIL directive in libc++ test These tests are failing in XCode 8.0, 8.1, and 8.2, but not in Xcode 8.3. Annoyingly the version numbering fo

Re: r296554 - [PS4] Set our default dialect to C++11. NFC for other targets.

On Wed, Mar 1, 2017 at 10:35 AM, Mehdi Amini wrote: > I’m not sure I find this nice to see this upstream. > > I not fond in general of this kind of difference in behavior. I don’t > think it is good for clang to have different default for this kind of > settings depending on the platform. It does

[PATCH] D27627: [WIP] Supporting C++ based kernel languages on AMDGPU Target

yaxunl updated this revision to Diff 90243. yaxunl edited the summary of this revision. https://reviews.llvm.org/D27627 Files: include/clang/AST/ASTContext.h include/clang/Basic/TargetInfo.h lib/AST/ASTContext.cpp lib/Basic/TargetInfo.cpp lib/Basic/Targets.cpp lib/CodeGen/CGBuiltin.cp

[PATCH] D29464: [MinGWToolChain] Don't use GCC headers on Win32

mati865 added a comment. Ping. https://reviews.llvm.org/D29464 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[PATCH] D30158: [clang-tidy] modernize: Find usage of random_shuffle and replace it with shuffle.

jroelofs added a comment. In https://reviews.llvm.org/D30158#690032, @madsravn wrote: > Looks good for the two tests the are for `random_shuffle` in llvm libc++. There were a lot more some time ago, before @mclow.lists performed this transformation on libc++'s testsuite. You might want to try

[PATCH] D30158: [clang-tidy] modernize: Find usage of random_shuffle and replace it with shuffle.

madsravn added a comment. Looks good for the two tests the are for `random_shuffle` in llvm libc++. https://reviews.llvm.org/D30158 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[PATCH] D30214: [Driver] Search for libc++ headers in ResourceDir

jroelofs added a comment. libc++ headers should not be installed in the resource dir. https://reviews.llvm.org/D30214 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[libcxx] r296685 - Generate the test configuration even when LIBCXX_INCLUDE_TESTS=OFF.

Author: ericwf Date: Wed Mar 1 15:53:30 2017 New Revision: 296685 URL: http://llvm.org/viewvc/llvm-project?rev=296685&view=rev Log: Generate the test configuration even when LIBCXX_INCLUDE_TESTS=OFF. This patch changes the CMake configuration so that it always generates the test/lit.site.cfg fil

[PATCH] D28348: [analyzer] Taught the analyzer about Glib API to check Memory-leak

zaks.anna accepted this revision. zaks.anna added a comment. This revision is now accepted and ready to land. > I am not clear why need to calculate the precise allocated size? The information generated by this checker is used for array bounds checking. For example, see https://reviews.llvm.org/

[PATCH] D30214: [Driver] Search for libc++ headers in ResourceDir

Hahnfeld added a comment. Ping https://reviews.llvm.org/D30214 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[PATCH] D30087: [Driver] Unify linking of OpenMP runtime

Hahnfeld added a comment. Ping https://reviews.llvm.org/D30087 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[PATCH] D30158: [clang-tidy] modernize: Find usage of random_shuffle and replace it with shuffle.

madsravn added a comment. In https://reviews.llvm.org/D30158#689904, @aaron.ballman wrote: > Out of curiosity, have you run this over libc++ or libstdc++ test suites > involving `std::random_shuffle`? If so, were the results acceptable? I haven't. Good idea. I will get onto that. I don't have

[PATCH] D30158: [clang-tidy] modernize: Find usage of random_shuffle and replace it with shuffle.

madsravn updated this revision to Diff 90223. madsravn added a comment. Last small changes based on comments. https://reviews.llvm.org/D30158 Files: clang-tidy/modernize/CMakeLists.txt clang-tidy/modernize/ModernizeTidyModule.cpp clang-tidy/modernize/ReplaceRandomShuffleCheck.cpp clang-

[PATCH] D30157: [analyzer] Improve valist check

zaks.anna added inline comments. Comment at: test/Analysis/valist-uninitialized-no-undef.c:25 + va_list va; + vprintf(isstring ? "%s" : "%d", va); //expected-warning{{Function 'vprintf' is called with an uninitialized va_list argument}} expected-note{{Function 'vprintf' is ca

[PATCH] D30341: [analyzer] clarify error messages about uninitialized function arguments

zaks.anna added inline comments. Comment at: lib/StaticAnalyzer/Checkers/CallAndMessageChecker.cpp:211 // Generate a report for this bug. - StringRef Desc = - describeUninitializedArgumentInCall(Call, IsFirstArgument); + std::string Desc = + des

[PATCH] D30489: [analyzer] catch out of bounds for VLA

zaks.anna added a comment. Gábor's suggestion sounds good to me. I think ArrayBoundCheckerV2 checker has a higher chance to be productized / moved out of alpha in the future. Should we just remove ArrayBoundChecker.cpp or is there a value in keeping it around? Repository: rL LLVM https://r

[PATCH] D30326: [MS-ABI] Allow #pragma section to choose for ZI data

rnk added a comment. This functionality doesn't feel worth a driver flag. It customizes a very small aspect of a Microsoft extension that was only implemented for compatibility. It also reimplements the logic that LLVM uses in the backend to put global variables in .bss or .data. Can you elabor

[PATCH] D30406: [Analyzer] Add support for displaying cross-file diagnostic paths in HTML output

zaks.anna added a comment. No multi-file support is a long outstanding limitation of scan-build html output. Great to see the patch!! Thank you for working on it! > It's not as immediately clear this is a multi-file output. In addition to Artem's suggestions, you might want to insert multiple

RE: r296554 - [PS4] Set our default dialect to C++11. NFC for other targets.

Probably better to debate this on the cfe-dev thread "Setting default dialect to C++11" than here. That said, it's far from the only target-dependent setting in the driver. There are things ranging from the default DWARF version to whether exceptions are on by default to whether Microsoft exten

[PATCH] D30158: [clang-tidy] modernize: Find usage of random_shuffle and replace it with shuffle.

aaron.ballman added a comment. Out of curiosity, have you run this over libc++ or libstdc++ test suites involving `std::random_shuffle`? If so, were the results acceptable? I think this generally LGTM (aside from the minor nit about naming), but I'd like to hear from @mclow.lists where this mee

r296664 - clang-format: [JS] Properly format object literals with shorthands.

Author: djasper Date: Wed Mar 1 13:47:28 2017 New Revision: 296664 URL: http://llvm.org/viewvc/llvm-project?rev=296664&view=rev Log: clang-format: [JS] Properly format object literals with shorthands. Before: return { a, b: 'b', c, }; After: return { a, b: 'b', c, };

[PATCH] D30158: [clang-tidy] modernize: Find usage of random_shuffle and replace it with shuffle.

madsravn added inline comments. Comment at: clang-tidy/modernize/ReplaceRandomShuffleCheck.cpp:77 + + auto Diag = [=]() { +std::string Message = ReplaceMessage; aaron.ballman wrote: > madsravn wrote: > > aaron.ballman wrote: > > > madsravn wrote: > > > > aar

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

4.0.1 it is. Thanks Hans! /Eric On Wed, Mar 1, 2017 at 10:08 AM, Hans Wennborg wrote: > 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.

[PATCH] D30158: [clang-tidy] modernize: Find usage of random_shuffle and replace it with shuffle.

aaron.ballman added inline comments. Comment at: clang-tidy/modernize/ReplaceRandomShuffleCheck.cpp:77 + + auto Diag = [=]() { +std::string Message = ReplaceMessage; madsravn wrote: > aaron.ballman wrote: > > madsravn wrote: > > > aaron.ballman wrote: > > >

[PATCH] D30158: [clang-tidy] modernize: Find usage of random_shuffle and replace it with shuffle.

madsravn added inline comments. Comment at: clang-tidy/modernize/ReplaceRandomShuffleCheck.cpp:77 + + auto Diag = [=]() { +std::string Message = ReplaceMessage; aaron.ballman wrote: > madsravn wrote: > > aaron.ballman wrote: > > > madsravn wrote: > > > > aar

r296659 - clang-format: [JS/TS] Properly understand cast expressions.

Author: djasper Date: Wed Mar 1 13:26:12 2017 New Revision: 296659 URL: http://llvm.org/viewvc/llvm-project?rev=296659&view=rev Log: clang-format: [JS/TS] Properly understand cast expressions. Many things were wrong: - We didn't always allow wrapping after "as", which can be necessary. - We used

[PATCH] D30268: Avoid copy of __atoms when char_type is char

hiraditya updated this revision to Diff 90214. hiraditya added a comment. Guarded the new implementation with _LIBCPP_ABI_OPTIMIZED_LOCALE macro to avoid ABI incompatibilities. https://reviews.llvm.org/D30268 Files: libcxx/include/__config libcxx/include/locale Index: libcxx/include/local

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

This revision was automatically updated to reflect the committed changes. Closed by commit rL296656: [PCH] Avoid VarDecl emission attempt if no owning module avaiable (authored by bruno). Changed prior to commit: https://reviews.llvm.org/D29753?vs=87774&id=90212#toc Repository: rL LLVM http

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

Author: bruno Date: Wed Mar 1 13:18:42 2017 New Revision: 296656 URL: http://llvm.org/viewvc/llvm-project?rev=296656&view=rev Log: [PCH] Avoid VarDecl emission attempt if no owning module avaiable This is a stopgap fix for PR31863, a regression introduced in r276159. Consider this snippet: str

[PATCH] D30158: [clang-tidy] modernize: Find usage of random_shuffle and replace it with shuffle.

aaron.ballman added inline comments. Comment at: clang-tidy/modernize/ReplaceRandomShuffleCheck.cpp:77 + + auto Diag = [=]() { +std::string Message = ReplaceMessage; madsravn wrote: > aaron.ballman wrote: > > madsravn wrote: > > > aaron.ballman wrote: > > >

[PATCH] D30158: [clang-tidy] modernize: Find usage of random_shuffle and replace it with shuffle.

madsravn marked an inline comment as done. madsravn added inline comments. Comment at: clang-tidy/modernize/ReplaceRandomShuffleCheck.cpp:77 + + auto Diag = [=]() { +std::string Message = ReplaceMessage; aaron.ballman wrote: > madsravn wrote: > > aaron.ballm

[PATCH] D29750: [PPC] Enable -fomit-frame-pointer by default for PPC

hfinkel accepted this revision. hfinkel added a comment. This revision is now accepted and ready to land. LGTM https://reviews.llvm.org/D29750 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe

[PATCH] D30158: [clang-tidy] modernize: Find usage of random_shuffle and replace it with shuffle.

aaron.ballman added inline comments. Comment at: clang-tidy/modernize/ReplaceRandomShuffleCheck.cpp:77 + + auto Diag = [=]() { +std::string Message = ReplaceMessage; madsravn wrote: > aaron.ballman wrote: > > Is there a reason this needs to capture everythin

Re: r296554 - [PS4] Set our default dialect to C++11. NFC for other targets.

I’m not sure I find this nice to see this upstream. I not fond in general of this kind of difference in behavior. I don’t think it is good for clang to have different default for this kind of settings depending on the platform. It does not provide a very good user experience from a cross-platfo

r296649 - Introduce an 'external_source_symbol' attribute that describes the origin

Author: arphaman Date: Wed Mar 1 12:06:25 2017 New Revision: 296649 URL: http://llvm.org/viewvc/llvm-project?rev=296649&view=rev Log: Introduce an 'external_source_symbol' attribute that describes the origin and the nature of a declaration This commit adds an external_source_symbol attribute to

[PATCH] D29819: Introduce an 'external_source_symbol' attribute that describes the origin and the nature of a declaration

This revision was automatically updated to reflect the committed changes. Closed by commit rL296649: Introduce an 'external_source_symbol' attribute that describes the origin (authored by arphaman). Changed prior to commit: https://reviews.llvm.org/D29819?vs=90042&id=90205#toc Repository: rL

[PATCH] D29819: Introduce an 'external_source_symbol' attribute that describes the origin and the nature of a declaration

arphaman added inline comments. Comment at: lib/Parse/ParseDeclCXX.cpp:3830-3837 + unsigned NumArgs; + // Some Clang-scoped attributes have some special parsing behavior. + if (ScopeName && ScopeName->getName() == "clang") +NumArgs = +ParseClangAttributeArgs(AttrNa

r296648 - [Test] NFC: Fixed typo in comments

Author: lcharles Date: Wed Mar 1 11:55:03 2017 New Revision: 296648 URL: http://llvm.org/viewvc/llvm-project?rev=296648&view=rev Log: [Test] NFC: Fixed typo in comments Changed "declerations" to "declarations" Modified: cfe/trunk/test/Modules/Inputs/merge-using-decls/b.h cfe/trunk/test/

r296646 - [analyzer] pr32088: Don't destroy the temporary if its initializer causes return.

Author: dcoughlin Date: Wed Mar 1 11:48:39 2017 New Revision: 296646 URL: http://llvm.org/viewvc/llvm-project?rev=296646&view=rev Log: [analyzer] pr32088: Don't destroy the temporary if its initializer causes return. In the following code involving GNU statement-expression extension: struct S

[PATCH] D30499: [analyzer] pr32088: Don't destroy the temporary if its initializer causes return.

This revision was automatically updated to reflect the committed changes. Closed by commit rL296646: [analyzer] pr32088: Don't destroy the temporary if its initializer causes… (authored by dcoughlin). Changed prior to commit: https://reviews.llvm.org/D30499?vs=90178&id=90201#toc Repository:

[PATCH] D30345: [CodeGen][Blocks] Refactor capture handling in code that generates block copy/destroy routines

arphaman updated this revision to Diff 90197. arphaman marked 4 inline comments as done. arphaman added a comment. Addressed John's comments. Repository: rL LLVM https://reviews.llvm.org/D30345 Files: lib/CodeGen/CGBlocks.cpp Index: lib/CodeGen/CGBlocks.cpp ===

LLVM buildmaster will be updated and restarted tonight

Hello everyone, LLVM buildmaster will be updated and restarted after 6 PM Pacific time today. Thanks Galina ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

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 wrote:

r296638 - [clang-format] Don't add namespace end comments for unbalanced right braces after namespace end

Author: krasimir Date: Wed Mar 1 10:38:08 2017 New Revision: 296638 URL: http://llvm.org/viewvc/llvm-project?rev=296638&view=rev Log: [clang-format] Don't add namespace end comments for unbalanced right braces after namespace end Modified: cfe/trunk/lib/Format/UnwrappedLineParser.cpp cf

[PATCH] D30499: [analyzer] pr32088: Don't destroy the temporary if its initializer causes return.

dcoughlin accepted this revision. dcoughlin added a comment. This revision is now accepted and ready to land. Seems like a safe quick fix for the crash. Looks good to me! https://reviews.llvm.org/D30499 ___ cfe-commits mailing list cfe-commits@lists

[PATCH] D30326: [MS-ABI] Allow #pragma section to choose for ZI data

thakis added a comment. Adding incompatible extensions to MS extensions sounds like bad precedent to me. I think it'd be good if a code owner could opine on this change before it goes in. https://reviews.llvm.org/D30326 ___ cfe-commits mailing lis

[clang-tools-extra] r296637 - [clangd] Unbreak the shared build.

Author: d0k Date: Wed Mar 1 10:23:40 2017 New Revision: 296637 URL: http://llvm.org/viewvc/llvm-project?rev=296637&view=rev Log: [clangd] Unbreak the shared build. Modified: clang-tools-extra/trunk/clangd/CMakeLists.txt Modified: clang-tools-extra/trunk/clangd/CMakeLists.txt URL: http://ll

[PATCH] D30498: [clangd] Add support for FixIts.

This revision was automatically updated to reflect the committed changes. Closed by commit rL296636: [clangd] Add support for FixIts. (authored by d0k). Changed prior to commit: https://reviews.llvm.org/D30498?vs=90181&id=90187#toc Repository: rL LLVM https://reviews.llvm.org/D30498 Files:

[clang-tools-extra] r296636 - [clangd] Add support for FixIts.

Author: d0k Date: Wed Mar 1 10:16:29 2017 New Revision: 296636 URL: http://llvm.org/viewvc/llvm-project?rev=296636&view=rev Log: [clangd] Add support for FixIts. Summary: This uses CodeActions to show 'apply fix' actions when code actions are requested for a location. The actions themselves make

[PATCH] D30326: [MS-ABI] Allow #pragma section to choose for ZI data

javed.absar added a comment. Hi Nico: This is for the case when using '#pragma bss_seg..' feature outside of msvc context. That's the reason for the flag. One could use attribute, but as attribute is attached to individual declaration, while pragma applies to whole file (unless superseded by an

[PATCH] D30498: [clangd] Add support for FixIts.

krasimir accepted this revision. krasimir added a comment. This revision is now accepted and ready to land. Looks Great! https://reviews.llvm.org/D30498 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/li

[PATCH] D30498: [clangd] Add support for FixIts.

bkramer updated this revision to Diff 90181. bkramer added a comment. - Use typedef instead of decltype() - Rename local variable not to shadow member. - Give FixIts their own mutex - Use const instead of let in typescript https://reviews.llvm.org/D30498 Files: clangd/ASTManager.cpp clangd/

[PATCH] D30111: [clang-format] Add a test to check at once all the Mozilla coding style

sylvestre.ledru added a comment. Manuel, is that ok with you? thanks (I will rename the file) https://reviews.llvm.org/D30111 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[PATCH] D30487: ClangFormat - Add option to break before inheritance separation operator in class declaration

djasper added a comment. Could you please upload a diff with the entire file as context? That makes reviewing this easier. Comment at: docs/ClangFormatStyleOptions.rst:428 +**BreakBeforeInhertianceDelimiter** (``boolt``) + If ``true``, in the class inheritance expression cl

[PATCH] D30405: [clang-format] Add a new flag FixNamespaceComments to FormatStyle

This revision was automatically updated to reflect the committed changes. Closed by commit rL296632: [clang-format] Add a new flag FixNamespaceComments to FormatStyle (authored by krasimir). Changed prior to commit: https://reviews.llvm.org/D30405?vs=90172&id=90179#toc Repository: rL LLVM h

r296632 - [clang-format] Add a new flag FixNamespaceComments to FormatStyle

Author: krasimir Date: Wed Mar 1 09:35:39 2017 New Revision: 296632 URL: http://llvm.org/viewvc/llvm-project?rev=296632&view=rev Log: [clang-format] Add a new flag FixNamespaceComments to FormatStyle Summary: This patch enables namespace end comments under a new flag FixNamespaceComments, which

[PATCH] D30498: [clangd] Add support for FixIts.

krasimir added inline comments. Comment at: clangd/ASTManager.cpp:105 + std::string Diagnostics; + decltype(this->FixIts) LocalFixIts; // Temporary storage + for (ASTUnit::stored_diag_iterator D = Unit->stored_diag_begin(), Why not typedef the type of the FixI

[PATCH] D30373: [analyzer] NFC: Update test infrastructure to support multiple constraint managers

ddcc added a comment. With the first method, I'm not sure how to refer to the `AnalyzerTest` class defined in `lit.cfg` from `lit.local.cfg`. It doesn't seem to be in scope, so unless I store an instantiation in the `config` object, I don't think it's accessible. Repository: rL LLVM https:

[PATCH] D30499: [analyzer] pr32088: Don't destroy the temporary if its initializer causes return.

NoQ created this revision. In the following code involving GNU statement-expression extension: struct S { ~S(); }; void foo() { const S &x = ({ return; S(); }); } function `foo()` returns before reference `x` is initialized. We shouldn't call the destructor for the temporary

[PATCH] D30487: ClangFormat - Add option to break before inheritance separation operator in class declaration

sylvestre.ledru added inline comments. Comment at: docs/ClangFormatStyleOptions.rst:430 + If ``true``, in the class inheritance expression clang-format will + break before operands ``:`` and ``,`` only if there is multiple + inheritance. s/is/are/ =

[PATCH] D30405: [clang-format] Add a new flag FixNamespaceComments to FormatStyle

djasper accepted this revision. djasper added a comment. This revision is now accepted and ready to land. LG https://reviews.llvm.org/D30405 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-c

[clang-tools-extra] r296627 - [clang-tidy] Attempt to fix the test where exceptions are disabled by default.

Author: alexfh Date: Wed Mar 1 08:41:11 2017 New Revision: 296627 URL: http://llvm.org/viewvc/llvm-project?rev=296627&view=rev Log: [clang-tidy] Attempt to fix the test where exceptions are disabled by default. Modified: clang-tools-extra/trunk/test/clang-tidy/modernize-use-override.cpp Mod

[PATCH] D30487: ClangFormat - Add option to break before inheritance separation operator in class declaration

Abpostelnicu updated this revision to Diff 90176. Abpostelnicu added a comment. added patch against the updated repo. Repository: rL LLVM https://reviews.llvm.org/D30487 Files: docs/ClangFormatStyleOptions.rst include/clang/Format/Format.h lib/Format/ContinuationIndenter.cpp lib/Form

[PATCH] D30498: [clangd] Add support for FixIts.

bkramer created this revision. This uses CodeActions to show 'apply fix' actions when code actions are requested for a location. The actions themselves make use of a clangd.applyFix command which has to be implemented on the editor side. I included an implementation for vscode. This also adds a -

[PATCH] D30476: [clangd] Add a toy Eclipse integration for development purposes

malaperle-ericsson added a comment. To be honest, I don't think this needs to be merged but it makes sense to put it somewhere people can try it. Repository: rL LLVM https://reviews.llvm.org/D30476 ___ cfe-commits mailing list cfe-commits@lists.

[clang-tools-extra] r296618 - [clangd] Add a toy VS Code integration for development purposes

Author: krasimir Date: Wed Mar 1 07:53:12 2017 New Revision: 296618 URL: http://llvm.org/viewvc/llvm-project?rev=296618&view=rev Log: [clangd] Add a toy VS Code integration for development purposes Summary: This patch adds bare-bone VS Code integration for development purposes of clangd. Revie

[PATCH] D30405: [clang-format] Add a new flag FixNamespaceEndComments to FormatStyle

krasimir updated this revision to Diff 90172. krasimir marked 2 inline comments as done. krasimir added a comment. - Address review comments https://reviews.llvm.org/D30405 Files: docs/ClangFormatStyleOptions.rst include/clang/Format/Format.h lib/Format/Format.cpp lib/Index/CommentToXML

[PATCH] D30489: [analyzer] catch out of bounds for VLA

xazax.hun added a comment. There is an alternative approach idea: This is not found by ArrayBoundCheckerV2? If no, an alternative approach would be to properly set the constraints on the extent of the VLA's memory region. After that, maybe ArrayBoundCheckerV2 would work automatically on this ca

[PATCH] D30487: ClangFormat - Add option to break before inheritance separation operator in class declaration

Abpostelnicu updated this revision to Diff 90171. Abpostelnicu added a comment. Also added tests. Repository: rL LLVM https://reviews.llvm.org/D30487 Files: docs/ClangFormatStyleOptions.rst include/clang/Format/Format.h lib/Format/ContinuationIndenter.cpp lib/Format/Format.cpp lib/

[PATCH] D27054: Introducing clang::tooling::AtomicChange for refactoring tools.

This revision was automatically updated to reflect the committed changes. Closed by commit rL296616: Introducing clang::tooling::AtomicChange for refactoring tools. (authored by ioeric). Changed prior to commit: https://reviews.llvm.org/D27054?vs=90018&id=90165#toc Repository: rL LLVM https

r296616 - Introducing clang::tooling::AtomicChange for refactoring tools.

Author: ioeric Date: Wed Mar 1 07:14:01 2017 New Revision: 296616 URL: http://llvm.org/viewvc/llvm-project?rev=296616&view=rev Log: Introducing clang::tooling::AtomicChange for refactoring tools. Summary: An AtomicChange is used to create and group a set of source edits, e.g. replacements or hea

  1   2   >